Fix `useless_format` suggestions

Auto merge of #3801 - Wilfred:patch-1, r=Manishearth

Fix broken markdown in README.md

Fix broken markdown in README.md

Auto merge of #3796 - rust-lang:clarify-readme, r=oli-obk

Clarify lint groups in readme

I'm explicitly avoiding mention of the deprecated/restriction lint groups, those exist more for testing purposes and are not really something people should be using.

r? @oli-obk @phansch

fixes #2964

Clarify lint groups in readme

Auto merge of #3795 - mikerite:test-rustfmt, r=phansch

Allow rustfmt to run on more tests

rustfmt tests/ui/methods.rs

Add `#[rustfmt::skip]` to methods tests

Many people run rustfmt automatically on save. Format-dependent tests
should be marked with `#[rustfmt::skip]` to prevent accidental
reformatting from this. As a bonus the rest of the code can the formatted.

Auto merge of #3789 - bzzzzzz:needless_range_loop_bugfix, r=oli-obk

Make needless_range_loop not applicable to structures without iter method

Fixes https://github.com/rust-lang/rust-clippy/issues/3788

Now we will start lint indexed structure only if it has known iter or iter_mut method implemented.

Fixed formatting and typo

Auto merge of #3781 - uniphil:write_with_newline_false_positive, r=oli-obk

Don't check [print/write]_with_newline on raw strings

Some tests for #3778 and some maybe-not-the-greatest code that passes those tests!

I didn't run `fmt` because a) it doesn't seem to install on nightly for me, and b) on stable it wanted to apply formatting to over 90 files. Happy to make any tweaks though!

I suspect this contribution may require more than just tweaks. I'm still sort of new to rust so it may not be idiomatic, and the specific approach I took feels a little heavy-handed and brittle. I'm happy to make changes with some guidance, or equally happy if this gives a starting place for someone else to do it better :)

Auto merge of #3779 - mikerite:fix-3704, r=phansch

Improve `iter_cloned_collect` suggestions

Fixes #3704

Make needless_range_loop not applicable to structures without iter method

Format again

Auto merge of #3776 - notriddle:drop-bounds, r=oli-obk

Add a lint to warn on `T: Drop` bounds

**What it does:** Checks for generics with `std::ops::Drop` as bounds.

**Why is this bad?** `Drop` bounds do not really accomplish anything.
A type may have compiler-generated drop glue without implementing the
`Drop` trait itself. The `Drop` trait also only has one method,
`Drop::drop`, and that function is by fiat not callable in user code.
So there is really no use case for using `Drop` in trait bounds.

**Known problems:** None.

**Example:**
```rust
fn foo<T: Drop>() {}
```

Fixes #3773

Auto merge of #3786 - rust-lang:rustup, r=Manishearth

Rustup to rustc master (32471f7ea 2019-02-19)

None

Rustup to rustc 1.34.0-nightly (32471f7ea 2019-02-19)

Format again

Add more descriptive details

Fix formatting

Check {print,write}_with_newline for literal newline

Both regular strings and raw strings can contain literal newlines. This commit
extends the lint to also warn about terminating strings with these.

Behaviour handling for raw strings is also moved into `check_newlines` by
passing in the `is_raw` boolean from `check_tts` as
[suggested](https://github.com/rust-lang/rust-clippy/pull/3781#pullrequestreview-204663732)

Auto merge of #3729 - illicitonion:match_enum_wildcard, r=flip1995

wildcard_enum_match_arm gives suggestions

And is also more robust

Fix Binding for rustc update

Restore tests

Also, fix existing test

wildcard_enum_match_arm gives suggestions

And is also more robust

Auto merge of #3783 - Enet4:patch-1, r=Manishearth

Update why transmute_int_to_float is bad

As suggested in #3550, this PR changes the reason why using `transmute` from an integer to a float is not recommended. Effectively, `from_bits` uses `transmute` underneath, but the former is preferred.

Don't fail for raw string ending in \n

Pass tests for #3778, {print,write}_with_newline false positive

This change guards the lint from checking newlines with a sort of complicated
check to see if it's a raw string. Raw strings shouldn't be newline-checked,
since r"\n" is literally \-n, not a newline. I think it's ok not to check for
_literal_ newlines at the end of raw strings, but maybe that's debatable.

I... don't think this code is that great. I wanted to write the check after
`check_tts`, but that was too late -- raw string type info is lost (or I
couldn't find it). Putting it inside `check_tts` feels heavy-duty and the check
itself feels like a brittle reach possibly into places it shouldn't.

Maybe someone can fix this up :)

Add failing test for #3778 write_with_newline

Literal `\n` characters (not a newline) in a `r"raw"` string should not
fail the lint.

This affects both write_with_newline and print_with_newline, so it is added in
both places.

I also copied a missing test case from write_with_newline over to
print_with_newline and added a note that one of those tests is supposed to
fail.

Format `transmute_from_int_to_float` docs

Update why transmute_int_to_float is bad

Fix #3550

Auto merge of #3777 - timotree3:patch-1, r=Manishearth

Fix typo in documentation for float_cmp_const lint

Format

Update README and CHANGELOG

Add a lint to warn on `T: Drop` bounds

**What it does:** Checks for generics with `std::ops::Drop` as bounds.

**Why is this bad?** `Drop` bounds do not really accomplish anything.
A type may have compiler-generated drop glue without implementing the
`Drop` trait itself. The `Drop` trait also only has one method,
`Drop::drop`, and that function is by fiat not callable in user code.
So there is really no use case for using `Drop` in trait bounds.

**Known problems:** None.

**Example:**
```rust
fn foo<T: Drop>() {}
```

Improve `iter_cloned_collect` suggestions

Fixes #3704

Add failing test for #3778 write_with_newline

Literal `\n` characters (not a newline) in a `r"raw"` string should not
fail the lint.

Add a semicolon as suggested

Fix typo in documentation for float_cmp_const lint

Auto merge of #3775 - flip1995:ice-3717, r=phansch

Fix ICE #3717 in lint implicit_hasher

Fixes #3717

This fixes the ICE. We lose some information in a very specific case though. But less information if better than an ICE. For an example see the test file.

Does anyone know, if there's another way to get the `ty::Ty` of a `hir::Expr`?

Add test for ICE #3717 fix

Fix ICE #3717

Restore the old `TypeckTables` in visit_body after walking the body

Auto merge of #3774 - notriddle:patch-1, r=flip1995

Use normal HTML label semantics for filter

The improvement is that you can click on the label and it'll select the form field.

Use normal HTML label semantics for filter

I legitimately don't understand why you did it with ARIA instead.

Auto merge of #3762 - mikerite:fix-build-20190214, r=flip1995

Fix breakage due to rust-lang/rust#58167

Fix breakage from rust-lang/rust#58296

Fix breakage due to rust-lang/rust#58167

Auto merge of #3760 - rust-lang:rustup, r=flip1995

Rustup

cc rust-lang/rust#58137

Rustup

cc rust-lang/rust#58137

Auto merge of #3756 - g-bartoszek:redundant-closure-for-methods, r=oli-obk

Redundant closure for methods

fixes  #3469

Auto merge of #3757 - mikerite:fix-2542, r=oli-obk

Fix `needless_range_loop` bad suggestion

Detect if the index variable is used inside a closure.

Fixes #2542

Auto merge of #3758 - phansch:document_macro, r=flip1995

Document `declare_clippy_lint` macro

Split up from my work on updating CONTRIBUTING.md, which is slowly
making progress.

cc #2666

Update comment regarding crate level cfg_attr

Document some more core functions

Document `declare_clippy_lint` macro

Split up from my work on updating CONTRIBUTING.md, which is slowly
making progress.

cc #2666

Fix `needless_range_loop` bad suggestion

Detect if the index variable is used inside a closure.

Fixes #2542

test formatting

Auto merge of #3754 - phansch:uicleanup, r=oli-obk

UI test cleanup: Extract similar_names tests

cc #2038

Auto merge of #3744 - phansch:fix3144, r=oli-obk

Fix ICE in needless_pass_by_value lint

If I understand it correctly, we were first creating a type with a
`RegionKind::ReErased` region and then deleted it again in
`util::implements_trait` with:

    cx.tcx.erase_regions(&ty);

causing the type query to fail.

It looks like using `ReEmpty` works around that deletion.

Fixes #3144

Auto merge of #3740 - flip1995:const_assert_macro, r=oli-obk

Macro check for assertion_on_constants lint

The `assertion_on_constants` lint currently has following output for this code [Playground](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=6f2c9df6fc50baf847212d3b5136ee97):
```rust
macro_rules! assert_const {
    ($len:expr) => {
        assert!($len > 0);
    }
}

fn main() {
    assert_const!(3);
    assert_const!(-1);
}
```
```
warning: assert!(const: true) will be optimized out by the compiler
 --> src/main.rs:3:9
  |
3 |         assert!($len > 0);
  |         ^^^^^^^^^^^^^^^^^^
...
8 |     assert_const!(3);
  |     ---------------- in this macro invocation
  |
  = note: #[warn(clippy::assertions_on_constants)] on by default
  = help: remove it
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assertions_on_constants

warning: assert!(const: false) should probably be replaced
 --> src/main.rs:3:9
  |
3 |         assert!($len > 0);
  |         ^^^^^^^^^^^^^^^^^^
...
9 |     assert_const!(-1);
  |     ----------------- in this macro invocation
  |
  = help: use panic!() or unreachable!()
  = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assertions_on_constants
```

This is contradictory. This lint should not trigger if the `assert!` is in a macro itself.

removing redundant closures in the whole project

Auto merge of #3755 - phansch:uicleanup2, r=oli-obk

UI test cleanup: Extract match_same_arms tests

cc #2038

redundant closure implemented for closures containing method calls

rustfmt

UI test cleanup: Rename copies.rs to match_same_arms.rs

UI test cleanup: Extract match_same_arms tests

UI test cleanup: Extract similar_names tests

Auto merge of #3751 - h-michael:opt-def-id, r=phansch

Use Hir::Def opt_def_id

related with: https://github.com/rust-lang/rust-clippy/pull/3750#issuecomment-462010225

Using `def.opt_def_id` in `opt_def_id` before replacing all all at once.

Use Hir::Def opt_def_id

Auto merge of #3748 - phansch:uitest, r=oli-obk

Add a uitest subcommand to simplify UI test invocation

This makes running single tests a lot easier.

It's now

`TESTNAME=xxx cargo uitest`

instead of

`TESTNAME=xxx cargo test --test compile-test`

Auto merge of #3750 - h-michael:const-params, r=phansch

Fix failing build  (adding new Hir::Def type ConstParam)

solves: #3749

related with:
- https://travis-ci.com/rust-lang/rust-clippy/jobs/176480646
- https://github.com/rust-lang/rust/pull/58191
- https://github.com/rust-lang/rust/commit/29f72063667187f2d281e940209a924730e165b2#diff-c1e317a81486d937bbfc6edca1dee92aR55

Add new Def type ConstParam

Add a uitest subcommand to simplify UI test invocation

This allows to run

`TESTNAME=xxx cargo uitest`

instead of

`TESTNAME=xxx cargo test --test-compile-test`

Auto merge of #3665 - jsgf:master, r=oli-obk

Start making clippy easier to invoke in non-cargo contexts

Clippy (clippy-driver) currently has a couple of strong but unnecessary couplings with cargo. This series:
1. makes detection of check builds more robust, and
2. make clippy-driver use the --sysroot specified on the command line as its internal sysroot.

Auto merge of #3743 - phansch:you_shall_not_pass, r=flip1995

Move run-pass tests to UI tests

This should give us more UI coverage for free.
It also removes the `run-pass` suite, so we now only have the `ui` suite.

Fix ICE in needless_pass_by_value lint

If I understand it correctly, we were first creating a type with a
`RegionKind::ReErased` region and then deleted it again in
`util::implements_trait` with:

    cx.tcx.erase_regions(&ty);

causing the type query to fail.

It looks like using `ReEmpty` works around that deletion.

Move run-pass tests to UI tests

This should give us more UI coverage for free.
It also removes the `run-pass` suite, so we now only have the `ui` suite.

Add tests for assertion_on_constants macro check

Don't warn about const assertions when assert is in a macro itself

Auto merge of #3725 - mikerite:fix-2728, r=phansch

Fix `cast_sign_loss` false positive

This checks if the value is a non-negative constant before linting about
losing the sign.

Because the `constant` function doesn't handle const functions, we check if
the value is from a call to a `max_value` function directly. A utility method
called `get_def_path` was added to make checking for the function paths
easier.

Fixes #2728

Document `get_def_path`

Merge pull request #3735 from jsgf/setup-toolchain

Add setup-toolchain.sh script to configure the master version of rustc,

Add setup-toolchain.sh script to configure the master version of rustc,
and update CONTRIBUTING.md accordingly.

Auto merge of #3737 - phansch:2018, r=phansch

Transition leftover test libs to Rust 2018

To tick off some checkboxes in https://github.com/rust-lang/rust/issues/58099 =)

Transition leftover test libs to Rust 2018

Auto merge of #3723 - rhysd:issue3721, r=oli-obk

Implement dbg_macro rule

Fixes  #3721

This patch adds new `dbg_macro` rule to check `dbg!` macro use.

Since this is my first patch to clippy, I'm not confident about following points:

- ~~Currently only checks `dbg!` span. Is it possible to check if the `dbg!` macro is provided by standard library or user-defined? If it's possible, I can make the check more strict.~~ Resolved as https://github.com/rust-lang/rust-clippy/pull/3723#discussion_r252417934
- ~~Is category `style` correct for this rule?~~'restriction' is used instead
- ~~Should I use `span_lint_and_sugg` instead of `span_lint`? Currently entire message is put as `msg`.  But later part `ensure to avoid having uses of it in version control` may be put as suggestion.~~ Done
- I'm not native English speaker. The message and doc may not be natural as English.

run `util/dev update_lints` and `cargo fmt --all`

Merge branch 'master' into issue3721

prefer `if` to `match`

use snippet for making a suggestion if possible

Auto merge of #3736 - mikerite:fix-build-20190203, r=phansch

Fix breakage due to rust-lang/rust#58079

The rustc change added HirId to a few nodes. As I understand it, the plan is
to remove the NodeId from these nodes eventually. Where the NodeId was
not being matched, I used `..` to try and avoid further breakage. Where it
was, I used `_` to make the fix easier when NodeId is removed.

Fix breakage due to rust-lang/rust#58079

The rustc change added HirId to a few nodes. As I understand it, the plan is
to remove the NodeId from these nodes eventually. Where the NodeId was
not being matched, I used `..` to try and avoid further breakage. Where it
was, I used `_` to make the fix easier when NodeId is removed.

Add a CI test for cargoless use of clippy-driver

base-tests: use subshells to manage current directory

It saves on having to pair `cd <path> && think && cd ..`.

Drive-by cleanups to cargo-clippy. No functional change.

Let CLIPPY_CONF_DIR be used to start search for config, and fall back to
CARGO_MANIFEST_DIR if it isn't set. If CARGO_MANIFEST_DIR isn't set, fall back
"." rather than panicing.

Issue #3663

clippy-driver: if --sysroot is specified on the command line, use that

If the user explicitly sets sysroot on the command line, then use that
value.

Issue #3663

clippy-driver: more robust test to see if we're clippy-enabled

Rather than looking for a fixed --emit arg set, just check to see
if we're emitting metadata at all. This makes it more robust to
being invoked by tools other than cargo (or if cargo changes its
invocation).

Issue #3663

Auto merge of #2857 - avborhanian:master, r=phansch

Adding lint test for excessive LOC.

This is a WIP for #2377. Just wanted to pull in because I had a few questions:

1. Is it okay that I'm approaching this via counting by looking at each line in the snippet instead of looking at the AST tree? If there's another way to do it, I want to make sure I'm doing the correct way, but I wasn't sure since the output AST JSON doesn't seem to contain whitespace.

2. My function is definitely going to trigger the lint, so also wanted to see if there was something obvious I could do to reduce it.

3. Are the two tests fine, or is there something obvious I'm missing?

4. Obviously bigger question - am I approaching the line count correctly. Current strategy is count a line if it contains some code, so skip if it's just comments or empty.

Auto merge of #3731 - phansch:travis, r=phansch

Travis: Don't run integration tests on every PR commit

This does not save Clippy any time but it makes sure that the concurrent
build limit is not reached as quickly for the `rust-lang` Travis
account.

I can't create a permalink to the discussion somehow, so here's an
excerpt from the Infra channel:

```
[11:57 PM] pietroalbini: there is a clippy build (20 jobs) and a packed_simd one (42 builders)  and a rustc one which isn't scheduling atm
[11:58 PM] pietroalbini: I don't think there is a way to prioritize rustc builds in the queue on travis, right?
[12:22 AM] alexcrichton: pietro: I don't think so no
[12:22 AM] alexcrichton: If it's a problem we should cull builds on other projects
[12:22 AM] alexcrichton: The rust repo is the #1 priority
```

Since the integration tests are rarely failing these days, I think it's
fine to not run them on every commit.

If needed, it's also still possible to do a complete test run with `@bors try`.

Auto merge of #3732 - phansch:fix_ice_3720, r=oli-obk

Fix ICE in vec_box lint and add run-rustfix

Includes https://github.com/rust-lang/rust-clippy/pull/3726

`hir::Ty` doesn't seem to know anything about type bounds and
`cx.tcx.type_of(def_id)` caused an ICE when it was passed a generic type
with a bound:

```
src/librustc_typeck/collect.rs:1311: unexpected non-type Node::GenericParam: Type { default: None, synthetic: None }
```

Converting it to a proper `Ty` fixes the ICE and catches a few more places
where the lint applies.

Fixes #3720