Auto merge of #5054 - flip1995:rustup, r=flip1995

Update custom_ice_message.stderr

changelog: none

Update custom_ice_message.stderr

Auto merge of #5034 - ThibsG:MatchWildErrArmImprove5024, r=flip1995

Match wild err arm improvements

This lint should trigger on other identifiers which have `_` prefix (such as `_e`) and only if they are unused in the panic block.

_Note_: the `is_unused` function is greatly inspired from `pat_is_wild` function in [loops lints](https://github.com/rust-lang/rust-clippy/blob/43ac9416d935942d6c7d2b2e0c876c551652c4ec/clippy_lints/src/loops.rs#L1689).
I've been considering doing some refactoring, maybe in utils. Maybe this PR or a new one. What do you think ?

fixes #5024

changelog: none

Auto merge of #5048 - justjavac:patch-1, r=llogiq

update license year to 2020

Happy new year 2020

changelog: none

update license year to 2020

Auto merge of #5023 - rust-lang:doc-main, r=phansch

Omit doc safety/errors header checking for main

This omits checking `main` methods, which are not usually documented in the way the others are.

changelog: none

Omit doc safety/errors header checking for main

Auto merge of #4543 - xiongmao86:issue4503, r=flip1995

Fix issue4503

Fixes #4503.

changelog: Add a lint checking user are using FileType.is_file() method and suggest using !FileType.is_dir().

- [x] Followed [lint naming conventions][lint_naming]
- [x] Added passing UI tests (including committed `.stderr` file)
- [x] `cargo test` passes locally
- [x] Executed `./util/dev update_lints`
- [x] Added lint documentation
- [x] Run `./util/dev fmt`

Auto merge of #5033 - JohnTitor:split-use-self, r=flip1995

Split up `use_self` ui test

Part of #2038

changelog: none

Extract visitor to utils

Change note message

Fix formatting

Better binding name on Err for note

Match underscore-prefixed variable also

Auto merge of #5042 - JohnTitor:rustup-0112, r=phansch

Rustup to rust-lang/rust#68045

This is blocked because `rustc_lint::context` is not pub module and `CheckLintNameResult` is not marked as `pub use`.

changelog: none

Rustup to rust-lang/rust#68045

Pull master, rebase, and update_lints again.

Update lints again.

Fix grammar error.

Update clippy_lints/src/methods/mod.rs

Co-Authored-By: Philipp Krones <hello@philkrones.com>

Change lint message.

./util/dev update_lints.

Fix documents and messages.

Update clippy_lints/src/methods/mod.rs

Extend spans to include !.

Fix lint warning in compile-test.rs

format codebase.

Add lint logic.

Declare lint.

Add test.

Split up `use_self` ui test

Auto merge of #5040 - JohnTitor:rustup-0111, r=flip1995

Some rustups

changelog: none

Remove references

Rustup to rust-lang/rust#67000

Comment out half-open patterns' tests

Apply review comments

Rustup to rust-lang/rust#67806

Apply review comments

Rustup to rust-lang/rust#68101

Rustup to rust-lang/rust#67258

Auto merge of #5031 - JohnTitor:editorconfig, r=phansch

Prefer 2 spaces as indent size over 4 on yaml

We use 2 spaces as an indent on YAML like `.travis.yml` now but `.editorconfig` tells my editor that indent size is 4, not 2. It makes my editor confused.

changelog: none

Auto merge of #5030 - JohnTitor:split-missing-doc, r=phansch

Split up `missing-doc` ui test

Part of #2038

changelog: none

Split up `missing-doc` ui test

Prefer 2 spaces as indent size over 4 on yaml

Auto merge of #4960 - ThibsG:patterns_with_wildcard_#4640, r=flip1995

New lint: pats_with_wild_match_arm

Wildcard use with other pattern in same match arm.

The wildcard covers other(s) pattern(s) as it will match anyway.

changelog: add new lint when multiple patterns (including wildcard) are used in a match arm.

Fixes #4640.

Auto merge of #5025 - JohnTitor:rustup-0109, r=flip1995

Some rustups

changelog: none

Rustup to rust-lang/rust#68024

Rustup to rust-lang/rust#67979

Rustup to rust-lang/rust#67781

Auto merge of #5022 - flip1995:useless_attr, r=phansch

Fix useless_attribute suggestion

Fixes #5021

changelog: Fix [`useless_attribute`] suggestion, which tripped rustfix

Add regression test for useless_attribute lint

Fix useless attribute suggestion

Auto merge of #5017 - sinkuu:mir_no_opt_fallout, r=flip1995

Fix redundant_clone lint not working with PathBuf and OsString

https://github.com/rust-lang/rust-clippy/pull/4825 diabled MIR optimization in clippy, including `rustc_mir::transform::InstCombine` which reduces `&(*x)` to `x`. This PR tries to unwrap `&*` when looking into `mir::Rvalue`s.

Fixes #5014.

---

changelog: fixed `redundant_clone` lint not working with `PathBuf` and `OsString`

Auto merge of #5015 - krishna-veerareddy:change-float-to-int-transmute-category, r=flip1995

Move `transmute_float_to_int` lint to `complexity`

`transmute_float_to_int` lint was accidentally added to nursery so moving it to the complexity group.

changelog: Move `transmute_float_to_int` out of nursery

Auto merge of #5019 - JohnTitor:follow-up-normalize, r=phansch

Normalize lint messages in cast_precision_loss

Follow-up of #5000

changelog: none

Normalize lint messages in cast_precision_loss

Auto merge of #5018 - JohnTitor:rustup-0108, r=phansch

Rustup to rust-lang/rust#67970

changelog: none

Rustup to rust-lang/rust#67970

Fix `redundant_clone`

Move `transmute_float_to_int` lint to `complexity`

`transmute_float_to_int` lint was accidentally added to nursery
so moving it to the complexity group.

Span help without suggestion

Change lint name to WILDCARD_IN_OR_PATTERNS

Handle case for non-exhaustive enums

Move to complexity and adapt test

 - test wildcard_enum_match_arm has been impacted by this new lint

Change group and use only func call

Remove useless parameters in func call

New lint: pats_with_wild_match_arm

 - Wildcard use with other pattern in same match arm

Auto merge of #5000 - JohnTitor:backticks, r=flip1995

Normalize lint messages

On rustc diagnostics, we prefer to use backticks over `'`,  `"`, or something else. I think we should follow their manner here.
In first commit, normalizes lint messages with backticks.
In second commit, updates all stderrs.
In third commit, updates descriptions on lintlist.

changelog: none

Bless 32bit test

Apply suggestion from code review

Update lintlist

Update stderrs

Normalize lint messages

Auto merge of #5011 - JohnTitor:split-collapsible-if, r=flip1995

Split up `collapsible_if` ui test

Part of #2038

changelog: none

Auto merge of #5010 - lzutao:recurse-remove_blocks, r=phansch

Make utils::remove_blocks non-recursive

changelog: none

Auto merge of #5002 - lzutao:fmt-fallout, r=flip1995

build: Use rustfmt from lastest nightly that contains it

changelog: none

Auto merge of #4999 - krishna-veerareddy:issue-4679-atomic-ordering, r=phansch

Add lint to detect usage of invalid atomic ordering

Detect usage of invalid atomic ordering modes such as `Ordering::{Release, AcqRel}` in atomic loads and `Ordering::{Acquire, AcqRel}` in atomic stores.

Fixes #4679

changelog: Add lint [`invalid_atomic_ordering`]

Split up `collapsible_if` ui test

Auto merge of #5009 - lzutao:travis-windows-msvc, r=phansch

Travis: Use windows-msvc target for Windows build

changelog: none
closes #5005

Make utils::remove_blocks non-recursive

Travis: Use windows-msvc target for Windows build

Prevent doc-tests from running and fix lint description

Split test cases into separate files

Add lint to detect usage of invalid atomic ordering

Detect usage of invalid atomic ordering modes such as
`Ordering::{Release, AcqRel}` in atomic loads and
`Ordering::{Acquire, AcqRel}` in atomic stores.

Auto merge of #5008 - JohnTitor:let-on-macros, r=flip1995

Do not trigger `let_and_return` lint on macros

Fixes #4997

changelog: Fix false positive in `let_and_return`

Do not trigger `let_and_return` lint on macros

build: Use rustfmt from lastest nightly that contains it

Auto merge of #5003 - JohnTitor:rustup, r=flip1995

Rustup to rust-lang/rust#67886

changelog: none

Fix test again

Pick up lost property

Rustup to rust-lang/rust#67886

Auto merge of #4998 - JohnTitor:revive-rls, r=llogiq

Re-enable rls integration test

It seems rls integration test passes correctly: https://travis-ci.com/rust-lang/rust-clippy/builds/143133541

changelog: none

Fix space size

Re-enable rls integration test

Auto merge of #4996 - JohnTitor:rustup, r=matthiaskrgr

Rustup to rust-lang/rust#67803 and rust-lang/rust#67137

changelog: none

Rustup to rust-lang/rust#67137

Rustup to rust-lang/rust#67803

Auto merge of #4993 - rust-lang:doc-unsafe-macro-check, r=flip1995

External macro check for missing_*_doc

This fixes #4949

changelog: none

External macro check for missing_*_doc

Auto merge of #4966 - bradsherman:iter-nth-zero, r=flip1995

New Lint: Iter nth zero

Check for the use of `iter.nth(0)` and encourage `iter.next()` instead as it is more readable

changelog: add new lint when `iter.nth(0)` is used

Fixes #4957

Add lint for iter.nth(0)

- Encourage iter.next() rather than iter.nth(0), which is less readable