Auto merge of #5788 - Leulz:shadow-unrelated-fix-rhs, r=flip1995

Removing RHS snippet from SHADOW_UNRELATED message.

Fixes #5703

I am not sure if I reinvented the wheel here, but I could not really find a snippet function that did this truncation, so I created the function. Please tell me if there was a more obvious way to do this, I am new here. :smile:

changelog: Truncates multi-line RHS in shadow_unrelated message if it has more than 5 lines.

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

Rustup

r? @ghost
changelog: none

Merge remote-tracking branch 'upstream/master' into rustup

Removing snippet from SHADOW_UNRELATED message.

WithOptConstParam::dummy -> WithOptConstParam::unknown

improve naming

const_eval_resolve

Remove lots of `Symbol::as_str()` calls.

In various ways, such as changing functions to take a `Symbol` instead
of a `&str`.

Auto merge of #5798 - mikerite:fix-2277-2, r=yaahc

Add test for `needless_range_loop` issue

Closes #2277

This was fixed when we fixed #2542.

changelog: none

Auto merge of #5793 - warner:5783-nth-zero-next, r=flip1995

improve advice in iter_nth_zero

fixes #5783

*Please keep the line below*
changelog:  For iter_nth_zero, the "use .next()" replacement advice is on the last line of the code snippet, where it is vulnerable to truncation. Display that advice at the beginning instead.

Auto merge of #5737 - Uriopass:unit-for-ord, r=flip1995

Reprise: new lint: Unintentional return of unit from closures expecting Ord

This lint catches cases where the last statement of a closure expecting
an instance of Ord has a trailing semi-colon. It compiles since the
closure ends up return () which also implements Ord but causes
unexpected results in cases such as sort_by_key.

Fixes #5080

Reprise of #5348 where I addressed all the comments there

changelog: add lint [`unit_return_expecting_ord`]

formatting nits

new lint: Returning unit from closures expecting Ord

This lint catches cases where the last statement of a closure expecting
an instance of Ord has a trailing semi-colon. It compiles since the
closure ends up return () which also implements Ord but causes
unexpected results in cases such as sort_by_key.

Fixes #5080

reprise: rebase, update and address all concerns

Merge commit '2ca58e7dda4a9eb142599638c59dc04d15961175' into clippyup

Auto merge of #5732 - bjorn3:patch-1, r=flip1995

Rename collapsable_if fix suggestion to "collapse nested if block"

The name "try" is confusing when shown as quick fix by rust-analyzer

changelog: Rename `collapsable_if` fix suggestion to "collapse nested if block"

Auto merge of #5773 - giraffate:repeat_once, r=flip1995

Add a lint for `.repeat(1)`

changelog: New lint `repeat_once`

fix #3028.

Add test for `needless_range_loop` issue

Closes #2277

This was fixed when we fixed #2542.

Use `.is_some()` not `Some(_)`

improve advice in iter_nth_zero

The "use .next()" replacement advice is on the last line of the code snippet,
where it is vulnerable to truncation. Display that advice at the beginning
instead.

closes #5783

Rename collapsable_if fix suggestion to "collapse nested if block"

The name "try" is confusing when shown as quick fix by rust-analyzer

Refactoring to use `constant_context

Use `constant_context`, `.is_str()` and `builtin_index()` to simplify.

Auto merge of #5792 - flip1995:rollup-torc1we, r=flip1995

Rollup of 5 pull requests

Successful merges:

 - #5443 (Some accuracy lints for floating point operations)
 - #5752 (Move range_minus_one to pedantic)
 - #5756 (unnecessary_sort_by: avoid linting if key borrows)
 - #5784 (Fix out of bounds access by checking length equality BEFORE accessing by index.)
 - #5786 (fix phrase in new_lint issue template)

Failed merges:

r? @ghost

changelog: rollup

Rollup merge of #5786 - matthiaskrgr:new_lint_issue_templ, r=phansch

fix phrase in new_lint issue template

changelog: none

Rollup merge of #5784 - matthiaskrgr:ice_5780, r=phansch

Fix out of bounds access by checking length equality BEFORE accessing by index.

Fixes #5780

changelog: fix out of bounds access in unnested_or_patterns lint.

Edit: I did not bother reducing a testcase from `librustc_typeck` crate but I can confirm that with the change the crash no longer occurs.

Rollup merge of #5756 - ebroto:5754_sort_by, r=flip1995

unnecessary_sort_by: avoid linting if key borrows

changelog: Avoid linting if key borrows in [`unnecessary_sort_by`]

Fixes #5754
Closes #2313

Rollup merge of #5752 - chrisduerr:pedantic_ranges, r=flip1995

Move range_minus_one to pedantic

This moves the range_minus_one lint to the pedantic category, so there
will not be any warnings emitted by default. This should work around
problems where the suggestion is impossible to resolve due to the range
consumer only accepting a specific range implementation, rather than the
`RangeBounds` trait (see #3307).

While it is possible to work around this by extracting the boundary into
a variable, I don't think clippy should encourage people to disable or
work around lints, but instead the lints should be fixable. So hopefully
this will help until a proper implementation checks what the range is
used for.

*Please keep the line below*
changelog: move [`range_minus_one`] to pedantic

Rollup merge of #5443 - thiagoarrais:issue-2040, r=flip1995

Some accuracy lints for floating point operations

This will add some lints for accuracy on floating point operations suggested by @clarfon in #2040 (fixes #2040).

These are the remaining lints:

- [x] x.powi(2) => x * x
- [x] x.logN() / y.logN() => x.logbase(y)
- [x] x.logbase(E) => x.log()
- [x] x.logbase(10) => x.log10()
- [x] x.logbase(2) => x.log2().
- [x] x * PI / 180 => x.to_radians()
- [x] x * 180 / PI => x.to_degrees()
- [x] (x + 1).log() => x.log_1p()
- [x] sqrt(x * x + y * y) => x.hypot(y)

changelog: Included some accuracy lints for floating point operations

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

Rustup

r? @ghost

changelog: none

fix phrase in new_lint issue template

Rollup merge of #72920 - oli-obk:const_transmute, r=RalfJung

Stabilize `transmute` in constants and statics but not const fn

cc #53605 (leaving issue open so we can add `transmute` to `const fn` later)

Previous attempt: #64011

r? @RalfJung

cc @rust-lang/wg-const-eval

Fix out of bounds access by checking length equality BEFORE accessing by index.

Fixes #5780

Stabilize `transmute` in constants and statics but not const fn

Auto merge of #5769 - robojumper:match_like_matches_macro, r=phansch

new lint: match_like_matches_macro

Suggests using the `matches!` macro from `std` where appropriate.

`redundant_pattern_matching` has been moved into the `matches` pass to allow suppressing the suggestion where `is_some` and friends are a better replacement.

changelog: new lint: `match_like_matches_macro`

Fix tests a bit more

Fix test failures

Fix typo

Run update_lints

Move range_minus_one to pedantic

This moves the range_minus_one lint to the pedantic category, so there
will not be any warnings emitted by default. This should work around
problems where the suggestion is impossible to resolve due to the range
consumer only accepting a specific range implementation, rather than the
`RangeBounds` trait (see #3307).

While it is possible to work around this by extracting the boundary into
a variable, I don't think clippy should encourage people to disable or
work around lints, but instead the lints should be fixable. So hopefully
this will help until a proper implementation checks what the range is
used for.

Avoid "whitelist"

Other terms are more inclusive and precise.

Auto merge of #5781 - giraffate:fix_a_broken_link, r=phansch

Fix a broken link in CONTRIBUTING.md

changelog: none

Fix a broken link in CONTRIBUTING.md

Auto merge of #5771 - montrivo:bugfix/single-match-else, r=matthiaskrgr

single_match_else - single expr/stmt else block corner case

One approach to fix #3489.
See discussion in the issue.

changelog: single_match_else - single expr/stmt else block corner case fix

single_match_else - single expr/stmt else block corner case

unnecessary_sort_by: avoid linting if key borrows

make match_like_matches_macro only apply to matches with a wildcard

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

Rustup

changelog: none

r? @ghost

Add a lint for `.repeat(1)`

fix #3028.

Auto merge of #74117 - Manishearth:rollup-ds7z0kx, r=Manishearth

Rollup of 14 pull requests

Successful merges:

 - #70563 ([rustdoc] Page hash handling)
 - #73856 (Edit librustc_lexer top-level docs)
 - #73870 (typeck: adding type information to projection)
 - #73953 (Audit hidden/short code suggestions)
 - #73962 (libstd/net/tcp.rs: #![deny(unsafe_op_in_unsafe_fn)])
 - #73969 (mir: mark mir construction temporaries as internal)
 - #73974 (Move A|Rc::as_ptr from feature(weak_into_raw) to feature(rc_as_ptr))
 - #74067 (rustdoc: Restore underline text decoration on hover for FQN in header)
 - #74074 (Fix the return type of Windows' `OpenOptionsExt::security_qos_flags`.)
 - #74078 (Always resolve type@primitive as a primitive, not a module)
 - #74089 (Add rust-analyzer to the build manifest)
 - #74090 (Remove unused RUSTC_DEBUG_ASSERTIONS)
 - #74102 (Fix const prop ICE)
 - #74112 (Expand abbreviation in core::ffi description)

Failed merges:

r? @ghost

Rollup merge of #73870 - sexxi-goose:projection-ty, r=nikomatsakis

typeck: adding type information to projection

This commit modifies the Place as follow:
* remove 'ty' from ProjectionKind
* add type information into to Projection
* replace 'ty' in Place with 'base_ty'
* introduce 'ty()' in `Place` to return the final type of the `Place`
* introduce `ty_before_projection()` in `Place` to return the type of
  a `Place` before i'th projection is applied

Closes https://github.com/rust-lang/project-rfc-2229/issues/5

Auto merge of #73978 - Mark-Simulacrum:shrink-paramenv, r=nnethercote

Shrink ParamEnv to 16 bytes

r? @nnethercote

x.py check passes but I haven't tried running perf or tests

Includes TODO for constants equivalent to π/180

Reclassify powi(2) lint under suboptimal_flops

Includes TODO comment for hypot lint

Place radian lints under suboptimal_flops

Better copy for lint message

Since x.log(y) is actually implemented as x.ln() / y.ln()

Lint for to_radians and to_degrees

fix match_like_matches_macro in clippy

Lint x.log(b) / y.log(b) => x.log(y)

new lint: match_like_matches_macro

Lint (x * x + y * y).sqrt() => x.hypot(y)

Lint for x.powi(2) => x * x

Auto merge of #5301 - JarredAllen:option_if_let_else, r=flip1995

Suggest `Option::map_or`(_else) for `if let Some { y } else { x }`

Fixes #5203

There are two issues with this code that I have noticed:

- Use of `Option::map_or` causes it to always evaluate the code in the else block. If that block is computationally expensive or if it updates some state (such as getting the next value from an iterator), then this change would cause the code to behave differently. In either of those circumstances, it should suggest Option::map_or_else, which takes both cases as a closure and runs one. However, I don't know how to check if the expression would change some state, so I left the lint's applicability as MaybeIncorrect.

- There are lints which can trigger on specific sub-cases of this lint (`if_let_some_result`, `question_mark`, and `while_let_loop`) and suggest different changes (usually better ones because they're more specific). Is this acceptable for clippy to give multiple suggestions, or should I have the code check if those other lints trigger and then not trigger this lint if they do?

changelog: Add lint [`option_if_let_else`]

Shrink ParamEnv to 16 bytes

Fixed compile errors

Formatted updates to lints

Update compile-test to follow new lint

Clean existing lint code to match new lint

Fix compile error from library change

Last few tweaks

Used clippy to clean itself

Don't lint if contains a macro

Refactoring pt. 2

Refactoring

Handle ref, mut, &, and &mut on the option

Properly parenthesize to avoid operator precedence errors

Stop linting on macros and correctly use braces for  constructs

Lint for if let Some(x) = ... instead of Option::map_or

Auto merge of #4841 - phaylon:pattern-type-mismatch, r=flip1995

Added restriction lint: pattern-type-mismatch

changelog: Added a new restriction lint `pattern-type-mismatch`. This lint is especially helpful for beginners learning about the magic behind pattern matching. (This explanation might be worth to include in the next changelog.)

fmt fix

LateContext has only one lifetime parameter now

Auto merge of #5761 - ThibsG:TypeRepetitionThreshold, r=flip1995

Improvements for `type_repetition_in_bounds` lint

Some improvements for `type_repetition_in_bounds`:
- add a configurable threshold to trigger the lint (#4380). The lint won't trigger anymore if there are more bounds (strictly) than `conf.max_trait_bounds` on this type.
- take generic args into account over bounded type (#4323)
- don't lint for predicates generated in macros (#4326)

Fixes #4380,
Fixes #4323,
Fixes #4326,
Closes #3764

changelog: Fix multiple FPs in `type_repetition_in_bounds` and add a configuration option

Note: the #3764 has already been fixed but not closed

Expanded lint documentation

Catching up with rustc changes

Fix rebase fallout

Adjusted expected STDERR

span_help_and_lint -> span_lint_and_help

Added restriction lint: pattern-type-mismatch

Don't lint for predicates generated in macros

Take generic args into account for bounded type

Add configurable threshold for `type_repetition_in_bounds` lint

Auto merge of #5763 - flip1995:rustup, r=Manishearth

Rustup

changelog: none

Merge remote-tracking branch 'upstream/master' into rustup

Use 'tcx for references to AccessLevels wherever possible.

rustc_lint: avoid using TypeckTables::empty for LateContext.

Auto merge of #5760 - phansch:deprecate-regex-macro, r=Manishearth

Deprecate regex_macro lint

Closes #2586

changelog: Deprecate regex_macro lint

Deprecate regex_macro lint

Auto merge of #5759 - mikerite:multiple_crate_versions_20200701, r=matthiaskrgr

Fix multiple_crate_versions error

Fix the versions of packages in the multiple_crate_versions ui test by
checking in the Cargo.lock for the test package. `ansi_term 0.11`
depends on `winapi ^0.3.4`. This means means that the expected stderr for
this test would have to be updated whenever `winapi 0.3` is updated
otherwise.

changelog: none

Fix multiple_crate_versions error

Fix the versions of packages in the multiple_crate_versions ui test by
checking in the Cargo.lock for the test package. `ansi_term 0.11`
depends on `winapi ^0.3.4`. This means means that the expected stderr for
this test would have to be updated whenever `winapi 0.3` is updated
otherwise.

Auto merge of #5758 - ebroto:5704_unnested_or_pats, r=flip1995

Require `or_patterns` to suggest nesting them

changelog: Require `#![feature(or_patterns)]` to trigger [`unnested_or_patterns`]

Fixes #5704

Require `or_patterns` to suggest nesting them