Auto merge of #5068 - JohnTitor:split-up-transmute, r=phansch

Split up `transmute` ui test

Part of #2038

changelog: none

Auto merge of #5028 - krishna-veerareddy:issue-5026-mem-ordering-fences, r=phansch

Detect usage of invalid atomic ordering in memory fences

Detect usage of `core::sync::atomic::{fence, compiler_fence}` with `Ordering::Relaxed` and suggest valid alternatives.

changelog: Extend `invalid_atomic_ordering` to lint memory fences

Fixes #5026

Decrease line length limit for stderr files

Auto merge of #5067 - JohnTitor:lint-skip-while-next, r=flip1995

Add `skip_while_next` lint

Fixes #4036

changelog: Add `skip_while_next` lint

Auto merge of #5070 - JohnTitor:suspicious-map-doc, r=flip1995

Improve `suspicious_map`documentation

Fixes #4793

changelog: none

Apply review comment

Auto merge of #5069 - JohnTitor:tweak-wording, r=flip1995

Tweak wording in `assertions_on_constants`

Displays actual macro names

changelog: none

Improve `suspicious_map`documentation

Tweak wording in `assertions_on_constants`

Split up `transmute` ui test

Remove trailing whitespaces

Add `skip_while_next` lint

Auto merge of #5056 - rust-lang:dissasociate-mut-key, r=flip1995

Avoid mut_key on types of unknown layout

This fixes #5020 by requiring a known layout for the key type before linting. Edit: This fixes #5043, too.

changelog: none

Avoid mut_key on types of unknown layout

Auto merge of #5032 - JohnTitor:add-sugg-some-result, r=flip1995

Add suggestions for `if_let_some_result`

Fixes #4991

This approach may be fragile though...

changelog: Add suggestions for `if_let_some_result`

Treat more strange pattern

Auto merge of #5066 - JohnTitor:split-up-1447, r=llogiq

Split up `if_same_then_else` ui test

Part of #2038

changelog: none

Split up `if_same_then_else` ui test

Apply review comments

Run `update_lints`

Reduce span range

Rename `ok_if_let` to `if_let_some_result`

Apply review comments

Add suggestion in `if_let_some_result`

Auto merge of #5065 - matthiaskrgr:rustup_28, r=matthiaskrgr

rustup https://github.com/rust-lang/rust/pull/67712

slice_patterns have been stabilized.

changelog: none

rustup https://github.com/rust-lang/rust/pull/67712

slice_patterns have been stabilized.

Auto merge of #5046 - JohnTitor:order-nonminimal-bool, r=flip1995

Keep the ordering in `nonminimal_bool` lint

I believe it shouldn't cause any regression but feel free to let me know if you have a doubtful example.
Also, splits up `booleans` ui test.

Fixes #5045

changelog: none

Auto merge of #5063 - JohnTitor:allow-correctly, r=flip1995

Allow `unused_self` lint at the function level

Another approach of #5062.
Fixes #5053

changelog: Allow `unused_self` lint at the function level

Allow `unused_self` lint at the function level

Auto merge of #5057 - rust-lang:pedantic_range_plus_one, r=flip1995

Downgrade range_plus_one to pedantic

This fixes #2217

changelog: Downgrade [`range_plus_one`] to `pedantic`

Downgrade range_plus_one to pedantic

Auto merge of #5059 - JohnTitor:rustup-0118, r=llogiq

Rustup to rust-lang/rust#68204

changelog: none

Rustup to rust-lang/rust#68204

Auto merge of #5055 - krishna-veerareddy:fix-new-lint-generator-import, r=llogiq

Fix new lint generator import

Fix `rustc_lint` import path and add lifetimes to `LateLintPass` in `new_lint` command.

changelog: none

Add lifetimes to `LateLintPass`

Fix rustc lint import paths generated by `new_lint`

Auto merge of #4994 - bradsherman:new_lint_gen, r=flip1995

Autogenerate new lints

Add option in clippy_dev to automatically generate boilerplate code for adding new lints

example:

`./util/dev new_lint --name=iter_nth_zero --type=late`

Fixes #4942

changelog: none

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

Update custom_ice_message.stderr

changelog: none

Update custom_ice_message.stderr

Add lint for default lint description

- Lint for any new lints that have the default lint description
  from the automation

Update documentation for adding new lints

- Add instructions for adding new lints with
  the new automation

Generate new lints easily

- Add option in clippy_dev to automatically generate boilerplate
  code for adding new lints

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

Split up `booleans` ui test

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`

Fix the ordering on `nonminimal_bool`

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

Add memory fence tests for `invalid_atomic_ordering`

Detect usage of invalid atomic ordering in memory fences

Detect usage of `core::sync::atomic::{fence, compiler_fence}`
with `Ordering::Relaxed` and suggest valid alternatives.

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