Auto merge of #5277 - chrissimpkins:rustc-dev-guide-transition, r=flip1995

Update rustc-guide to rustc-dev-guide

The rustc-guide is being renamed to the rustc-dev-guide. The discussion is in rust-lang/rustc-guide#470.

This PR revises rustc-guide to rustc-dev-guide in the Readme Markdown file.

Transition tracker: rust-lang/rustc-guide#602

changelog: none

update rust-lang.github.io to rustc-dev-guide.rust-lang.org

Auto merge of #5292 - jpospychala:map-placeholder, r=flip1995

Improve placeholder in map_unit_fn

Instead of using `_` as a default placeholder use `a`.

fixes #5180
changelog: Improve default placeholder in map_unit_fn

Improve placeholder in map_unit_fn

Auto merge of #5287 - matthiaskrgr:pat_isref, r=flip1995

redundant_pattern: take binding (ref, ref mut) into account in suggestion

fixes #5271

changelog: redundant_pattern: take binding (ref, ref mut) into account in suggestion (#5271)

Auto merge of #5286 - matthiaskrgr:delay_vec_creation, r=flip1995

check_pat: delay creation of the "normal" vec until we reach the branch where it is actually needed

changelog: none

Auto merge of #5291 - ThibsG:FixSingleBinding, r=flip1995

Fix match single binding when in a let stmt

Fix bad suggestion when `match_single_binding` lints when inside a local (let) statement.

Fixes #5267

changelog: none

Fix match single binding when in a let stmt

Auto merge of #5285 - matthiaskrgr:dep_it, r=flip1995

deps: bump itertools 0.8 -> 0.9

changelog: none

Auto merge of #5290 - polybuildr:map-message, r=phansch

Improve error messages for {option,result}_map_unit_fn

Instead of saying "unit function", use the phrase the description uses: "function that returns the unit type".

Fixes #5180.

changelog: Improve error messages for {option,result}_map_unit_fn

Auto merge of #5289 - polybuildr:setup, r=matthiaskrgr

Mention the setup instructions in CONTRIBUTING

Multiple people seem to have missed this based on questions asked in the Discord channel. Might as well mention it up front.

changelog: none

Improve error messages for {option,result}_map_unit_fn

Instead of saying "unit function", use the phrase the description
uses: "function that returns the unit type".

Fixes #5180.

Mention the setup instructions in CONTRIBUTING

Multiple people seem to have missed this based on questions asked in the
Discord channel. Might as well mention it up front.

redundant_pattern: take binding (ref, ref mut) into account in suggestion.

fixes #5271

check_pat: delay creation of the "normal" vec until we reach the branch where is is actually needed

deps: bump itertools 0.8 -> 0.9

changelog: none

transition rustc-guide to rustc-dev-guide

Auto merge of #5276 - flip1995:macro_use, r=flip1995

Rename macro_use_import -> macro_use_imports

I missed this during review of #5230. We can just do this, without deprecating the old name, since this lint didn't hit nightly rustc yet.

changelog: none

Rename macro_use_import -> macro_use_imports

Auto merge of #5230 - DevinR528:macro-use, r=flip1995

Macro use

---

changelog: This lint enforces Rust 2018 idiom of importing macro's directly without `#[macro_use]` fixes #5179 .

Auto merge of #5235 - flip1995:tag_deploy_fix, r=phansch

Fix deploy script for tag deploys

By hand updating the documentation for the `rust-1.41.1` release, I found, that the deploy script was buggy for tag deploys. Luckily we weren't affected yet (will only run if the commit of the tag already has the workflow config, so probably ~`rust-1.43.0`)

The bug:

When a tag was deployed, until now it also updated the master documentation to the state of the tag. This was because the `git add .` not only updated the files created or modified by the tag commit, but also the master lints.json file.

changelog: none

Auto merge of #5257 - mlegner:cast_hex_fp, r=flip1995

Resolve false positives of unnecessary_cast for non-decimal integers

This PR resolves false positives of `unnecessary_cast` for hexadecimal integers to floats and adds a corresponding test case.

Fixes: #5220
changelog: none

warn on macro_use attr

Auto merge of #5266 - sinkuu:questionmark, r=flip1995

Lint `if let Some` and early return in question_mark lint

Fixes #5260

changelog: lint `if let Some` and early return in `question_mark` lint

Apply suggestions from code review

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

Simplify if_chain.

Move NumericLiteral to its own module.

Included binary and octal cases.

Resolve false positives for hex int cast.

Test for unnecessary_cast of hex int literal.

Auto merge of #5258 - ThibsG:UselessBindingInStruct638, r=flip1995

Add lint for .. use in fully binded struct

This PR adds the lint `match-wild-in-fully-binded-struct` to prevent the use of the `..` pattern when all fields of the struct are already binded.

Fixes: #638
changelog: Add [`rest_pat_in_fully_bound_structs`] lint to warn against the use of `..` in fully binded struct

Auto merge of #5246 - JarredAllen:master, r=flip1995

Detect usage of custom floating-point abs implementation

Closes #5224

changelog: Enhance [`suboptimal_flops`] lint to detect manual implementations of the `abs` method

run-rustfix

Lint `if let Some` in question_mark lint

Add restrictive pat use in full binded struct

Update test case answers to match cargo dev fmt

Ran cargo dev fmt

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

Rustup to rust-lang/rust#69506

changelog: none

Rustup to rust-lang/rust#69506

Recommended changes from flip1995

Auto merge of #5262 - JohnTitor:rls, r=flip1995

Revive rls integration test

RLS updated their dependencies in rust-lang/rls#1646 so we can revive the integration test. I confirm it passed on my local.

changelog: none

Revive rls integration test

Auto merge of #5261 - matthiaskrgr:qmark, r=flip1995

use question mark operator

changelog: none

use question mark operator

Auto merge of #5256 - JohnTitor:try-eval-usize, r=phansch

Use `try_eval_usize` over `eval_usize`

Fixes #5223

changelog: Fix ICE in evaluating usizes

Add regression test

Use `try_eval_usize` over `eval_usize`

Auto merge of #5259 - flip1995:lang_items, r=phansch

Use lang items instead of get_trait_def_id where possible

changelog: none

Auto merge of #5047 - flip1995:use_debug, r=phansch

Don't trigger use_debug lint in Debug impl

Fixes #5039

changelog: Don't trigger [`use_debug`] lint in Debug impl

Auto merge of #5250 - mlegner:unused_fp, r=flip1995

Whitelist unused attribute for use items.

This PR whitelists the `unused` attribute with `use` items and adds a corresponding test case.

Fixes: #5229
changelog: none

Add path for display trait

Use lang items instead of get_trait_def_id where possible

Update stderr

Don't lint debug formatting in debug impl

Auto merge of #5242 - matthiaskrgr:5238_test, r=flip1995

add test for #5238

changelog: none

Auto merge of #5254 - JohnTitor:sugg, r=flip1995

Rustup to rust-lang/rust#69469

changelog: none

Rustup to rust-lang/rust#69469

Whitelist unused attribute for use items.

Fix one last test issue

Refactor suggested by krishna-veerareddy

Fixed compile error from merging

Merge branch 'master' of github.com:rust-lang/rust-clippy

Auto merge of #5247 - JohnTitor:syntax, r=flip1995

Some rustups

changelog: none

Disable rls test for now

Rustup to rust-lang/rust#69579

Rustup to rust-lang/rust#69592

Changed test output to reflect cargo fmt

Run cargo dev fmt

Finished checking for cases of absolute values

add test for #5238

Some bugfixing

Created floating point abs lint and test, but not yet run

Fix deploy script for tag deploys

Auto merge of #5232 - JohnTitor:git2-0.12, r=phansch

Update git2 to 0.12

changelog: none

Update git2 to 0.12

Auto merge of #5231 - JohnTitor:associated-type-defaults, r=phansch

Rustup to rust-lang/rust#61812

changelog: none

Rustup to rust-lang/rust#61812

Auto merge of #4897 - krishna-veerareddy:issue-2040-accurate-float-functions, r=flip1995

Add lint to improve floating-point expressions

Looks for floating-point expressions that can be expressed using built-in methods to improve accuracy, performance and/or succinctness.

changelog: Add lint `floating_point_improvements`.

Fixes #4726
Partly addresses [#2040](https://github.com/rust-lang/rust-clippy/issues/2040)

Currently linted expressions:

| Expression | Suggestion |
|---------------------------------|------------|
| x.log(2.0) | x.log2() |
| x.log(10.0) | x.log10() |
| x.log(std::f32::consts::E) | x.ln() |
| (1 + x).ln() | x.ln_1p() |
| (2.0).powf(x) | x.exp2() |
| (std::f32::consts::E).powf(x) | x.exp() |
| x.powf(1/2) | x.sqrt() |
| x.powf(1/3) | x.cbrt() |
| x.powf(y), where y is whole | x.powi(y) |
| x.exp() - 1 | x.exp_m1() |
|x * y + z|x.mul_add(y, z)|

Auto merge of #5222 - JohnTitor:dont-deploy, r=flip1995

Do not run deploy action on other repos

Usually, we don't have to run deploy action on other repos, let's ignore there.

changelog: none

Add `imprecise_flops` lint

Add lint to detect floating point operations that can be computed more
accurately at the cost of performance. `cbrt`, `ln_1p` and `exp_m1`
library functions call their equivalent cmath implementations which is
slower but more accurate so moving checks for these under this new lint.

Rename `mul_add` test file and add general improvements

Move `manual_mul_add` into `suboptimal_flops` lint

Run rust-fix on tests

Fix `powi` suggestion and add general improvements

Change lint name to `suboptimal_flops`

Remove lint for logarithm division identity

Add type suffixes to unsuffixed method receiver suggestions

Detect usage of `(x + 1).ln()` and suggest `x.ln_1p()` instead

Split test cases into separate files

Lint expressions of the form `x.log(b) / y.log(b)`

Suggest usage of `powi` method when applicable

Consolidate the accuracy and efficiency lints

Merge the accuracy and efficiency lints into a single lint that
checks for improvements to accuracy, efficiency and readability
of floating-point expressions.

Add lints to detect inaccurate and inefficient FP operations

Add lints to detect floating point computations that are either
inaccurate or inefficient and suggest better alternatives.

Do not run deploy action on other repos

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

Rustup to rust-lang/rust#69366

changelog: none

Rustup to rust-lang/rust#69366

Auto merge of #5216 - krishna-veerareddy:issue-5192-fp-const-fn, r=flip1995

Prevent `missing_const_for_fn` on functions with const generic params

`const` functions cannot have const generic parameters so prevent the
`missing_const_for_fn` lint from firing in that case.

changelog: Fix false positive in `missing_const_for_fn`

Fixes #5192

Prevent `missing_const_for_fn` on functions with const generic params

`const` functions cannot have const generic parameters so prevent the
`missing_const_for_fn` lint from firing in that case.

Auto merge of #5029 - flip1995:wildcard_imports, r=phansch

New Lint: `wildcard imports`

Fixes #1228

### A few notes:
- I put this lint in the pedantic group, even though in the issue restriction was mentioned.
- Every fallout fix was automatically applied by `cargo fix` (This produced 3 `unused_imports` warnings) and are in commit 7e834c8. So reverting these changes wouldn't be a problem.

### A few ideas:
- A configuration to specify the amount of imported Items, where a `*` might be better.
- ~~A configuration to disable the lint for enums. Or just disable the lint for enums, since there is [`enum_glob_use`](https://rust-lang.github.io/rust-clippy/master/index.html#enum_glob_use)~~ I moved `enum_glob_use` into this lint in 12937f0

### A few quotes from the issue:
> Is there a way to ask the compiler about the modules or symbols that the current file is using?

Yes there is. I found it, once I was nearly finished implementing it myself. See 321d64a

> one hard optional feature would be to figure out what is currently used and add a suggestion to replace it with a full import list.

Yeah that was pretty hard, until I found the query for this. Queries are cool, but too hard to find.

> FWIW VS Code and Intellij IDEA both offer imports deglobbing which replace * with required imports.

And now, Clippy can do this too! :tada:

---

Your thoughts on the notes/ideas?

changelog: Add new lint [`wildcard imports`]. Add suggestion to [`enum_glob_use`]

Fix suggestion for weird formattings