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

Auto merge of #5318 - matthiaskrgr:rustup_30, r=flip1995

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

changelog: none

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

Auto merge of #5316 - lzutao:osstr, r=phansch

Don't convert Path to lossy str

changelog: none

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

Rustup to rust-lang/rust#69076

changelog: none

Rustup to rust-lang/rust#69076

Don't convert Path to lossy str

Use `into_path`

Use pattern matching instead of manually checking condition

Auto merge of #5312 - joshtriplett:wildcard-prelude-documentation, r=flip1995

Document that wildcard_imports doesn't warn about `use ...::prelude::*;`

changelog: Improve [`wildcard_imports`] documentation

Auto merge of #5314 - ehuss:remove-git2, r=flip1995

Remove git2 dependency.

This removes the `git2` dependency (used in the integration test).  Updating git2 is awkward because both cargo and clippy have to be updated in sync, so this removes that requirement. It didn't look like it was using the git2 library for any particular reason, so this just launches the `git` executable, which should be available more or less everywhere.

This unblocks updating Cargo.

changelog: none

Fix typo

Co-Authored-By: Mateusz Mikuła <mati865@users.noreply.github.com>

Remove git2 dependency.

Document that wildcard_imports doesn't warn about `use ...::prelude::*;`

Auto merge of #5308 - flip1995:changelog, r=Manishearth

Update Changelog

I also changed the formatting of the changelog for the beta changelog. I can also do this for all the previous releases, if this formatting gets accepted.

[Rendered](https://github.com/flip1995/rust-clippy/blob/changelog/CHANGELOG.md)

r? @Manishearth

changelog: none

Auto merge of #5304 - sinkuu:redundant_clone_not_consumed, r=flip1995

Extend `redundant_clone` to the case that cloned value is not consumed

Fixes #5303.

---

changelog: Extend `redundant_clone` to the case that cloned value is not consumed

Auto merge of #5307 - JohnTitor:rename-method, r=flip1995

Rustup to rust-lang/rust#69674

changelog: none

Change changelog formatting

Update changelog_update doc to reflect the actual ordering of the changelog

Update CHANGELOG.md

Rustup to rust-lang/rust#69674

Use visit_place

Check for mutation

Only fires on temporaries

`let y = x.clone()` cannot be turned into `let y = x` without moving x,
regardless of whether `y` is consumed or not.

Extend `redundant_clone` to the case that cloned value is not consumed

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