Add `rustfmt::skip` to `double_parens` tests

Auto merge of #3666 - detrumi:map-or-on-non-copy, r=flip1995

Only suggest map_or for copy types

Fixes #2686

Auto merge of #3826 - mikerite:rustfmt-outer-attr-tests, r=phansch

Add `rustfmt::skip` to outer attr format tests

Auto merge of #3827 - phansch:rustup, r=Manishearth

Rustup rust-lang/rust#58321

Fixes the build after https://github.com/rust-lang/rust/issues/58321

Rustup https://github.com/rust-lang/rust/pull/58321

Add `rustfmt::skip` to outer attr format tests

Auto merge of #3821 - g-bartoszek:redundant_closure-different-borrow-levels, r=oli-obk

do not trigger redundant_closure when there is a difference in borrow…

… level between closure parameter and "self", fixes  #3802

Auto merge of #3818 - phansch:update_changelog, r=oli-obk

Update changelog for Rust 1.33

None

Fix false negative

Attempt to fix false negative

Move lint_map_unwrap_or to its own file

Only suggest map_or for copy types

Auto merge of #3817 - rust-lang:fix-bool_comparison-on-non-bool, r=oli-obk

Fix `bool_comparison` with non-`bool` expressions

Fixes #3703.
It just moves around the type check that was already there for some comparison to all of them, because if one type isn't `bool`, none of those comparison can be simplified.

do not trigger redundant_closure when there is a difference in borrow level between closure parameter and "self"

Auto merge of #3814 - ljedrz:HirIdification_lockstep_upgrade, r=phansch

HirIdify some lints

Unblocks https://github.com/rust-lang/rust/pull/58561 (a part of [rust-lang/rust#57578](https://github.com/rust-lang/rust/pull/57578)). Can we branch it like with https://github.com/rust-lang/rust-clippy/pull/3790? I can rebase on a different commit if need be.

Haven't had time to run tests yet, so I'd wait for Travis 🙈.

fix line format

Auto merge of #3819 - mikerite:rustfmt-formatting-tests, r=phansch

Add [rustfmt::skip] to formatting lint tests

Auto merge of #3820 - mikerite:remove-feature-try-from, r=Manishearth

Remove `#[feature(try_from)]`

`try_from` is now stable.

Remove `#[feature(try_from)]`

`try_from` is now stable.

Add [rustfmt::skip] to formatting lint tests

Update changelog for Rust 1.33

Fix `bool_comparison` with non-`bool` expressions

Auto merge of #3808 - mikerite:useless-format-suggestions, r=oli-obk

Fix `useless_format` suggestions

Auto merge of #3816 - g-bartoszek:redundant_closure_for_macros, r=Manishearth

do not trigger redundant_closure for external macros

fixes #3791

redundant_closure does not trigger for external macros

Auto merge of #3772 - flip1995:ice-3719, r=Manishearth

Fix ICE #3719+#3718 in lint match_ref_pats

Fixes #3719
This conveniently also fixes #3718

The ICE occurs when the match expression was a macro call, where the macro was defined in another file. Since we don't have the ability to reproduce this behavior with our UI tests (AFAIK), I couldn't add a test reproducing this ICE.. However, I added a test which is related to the ICE, to show the new behavior of the lint.

I tested it with the mscheme repo locally and the ICE didn't happen anymore.

r? @matthiaskrgr

Auto merge of #3805 - martinsp:ice-3747, r=Manishearth

Fix ICE #3747

I'm not sure if this was the correct approach.

I don't know if I put tests/ui/crashses/ice-3747.rs in correct place because the test always passed when I ran it with `cargo test`, even without the fix applied.

If I run that test with `env CLIPPY_TESTS=true cargo run --bin clippy-driver -- -L ./target/debug tests/ui/crashes/ice-3747.rs` then the test correctly fails without the fix applied

fixes #3747

fix: replace wrong id

Auto merge of #3815 - phansch:make_private, r=Manishearth

Make DiagnosticWrapper private

It's doesn't have to be public anywhere outside of clippy_lints as far as I can tell.

Make DiagnosticWrapper private

It's doesn't have to be public anywhere outside of clippy_lints

HirIdify some lints

Auto merge of #3811 - rust-lang:test-for-2526, r=mcarton

Add a test for #2526

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.

Add a test for #2526

Auto merge of #3810 - phansch:refactor/extract_module, r=flip1995

Extract diagnostics module and document some functions

This moves the lint building functions from `utils/mod.rs` to their own
`utils/diagnostics.rs` file. Also adds documentation for three of them.

Auto merge of #3790 - ljedrz:HirIdify_intravisit, r=phansch

partially HirIdify lints

Enables https://github.com/rust-lang/rust/pull/58232 (a part of https://github.com/rust-lang/rust/pull/57578).

fix formatting

Extract diagnostics module and document some functions

This moves the lint building functions from `utils/mod.rs` to their own
`utils/diagnostics.rs` file. Also adds documentation for three of them.

Document some `span_lint_*` util functions

Fix `useless_format` suggestions

Fix ICE #3747

[Martins Polakovs, John Firebaugh]

Auto merge of #3798 - eyko:patch-1, r=flip1995

Fix broken link in README

I noticed the link to the lint list had a typo, so this PR is just that:

```diff
- [lint list]](https://rust-lang.github.io/rust-clippy/master/index.html)
+ [lint list](https://rust-lang.github.io/rust-clippy/master/index.html)
```

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

Fix broken markdown in README.md

Fix broken markdown in README.md

Fix broken link in README

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 :)

partially HirIdify lints

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

Document the new `Sugg` functions

Add test related to the ICE

This test doesn't reproduce the ICE since it only happens, when the macro is defined in another file.
Currently we can't add tests with multiple files AFAIK

Also using the auxiliary folder didn't help

Fix ice-3719

Implement Sugg::hir_with_macro_callsite

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