Auto merge of #5272 - jmeyers35:file_read_lint, r=flip1995

add lint on File::read_to_string and File::read_to_end

Adds lint `verbose_file_reads` which checks for use of File::read_to_end and File::read_to_string.

Closes https://github.com/rust-lang/rust-clippy/issues/4916

changelog: add lint on File::{read_to_end, read_to_string}

Auto merge of #5300 - JohnTitor:edition-flag, r=flip1995

Use `edition:2018` flag more widely

Now we recommend using `// edition:2018`, so let's use it more widely.
Also replace a too old example with new one in the docs.

changelog: none

add CR feedback

Improve documentation

Use `edition:2018` flag more widely

Auto merge of #5298 - rust-lang:needless_doc_main_code, r=flip1995,Manishearth

needless_doc_main: only check rust code

This fixes #5280 by checking the language attribute on code blocks.

---

changelog: none

Update tests/ui/needless_doc_main.rs

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

Auto merge of #5299 - JohnTitor:contrib, r=flip1995

Clean-up docs

Fixes #5268

changelog: none

Move links to the end of each chapter on adding_lints

Move links to the end of each chapter on CONTRIBUTING

Clean-up adding_lints.md

Clean-up CONTRIBUTING.md

needless_doc_main: only check rust code

Auto merge of #5296 - sinkuu:fix_ice_trivial_bounds, r=flip1995

Fix ICE with trivial_bounds feature

https://github.com/rust-lang/rust/issues/69874#issuecomment-596890446

changelog: Fix ICE with trivial_bounds feature

Auto merge of #5297 - JohnTitor:shadow, r=phansch

Use `node_type_opt` over `node_type`

Fix ICE reported in rust-lang/rust#69873.

changelog: Fix ICE in checking bindings

Use `node_type_opt` over `node_type`

Fix doc

Fix ICE with trivial_bounds feature

Auto merge of #5295 - matthiaskrgr:lints_readme, r=flip1995

clippy_lints: readme: don't mention crates.io since it is no longer used to publish clippy

changelog: none

clippy_lints: readme: don't mention crates.io since it is no longer used to publish clippy.

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

add lint on File::read_to_string and File::read_to_end

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