rustup "Add span information to `ExprKind::Assign`"

Auto merge of #4946 - lzutao:take, r=flip1995

a few minor cleanups

changelog: none

do minor cleanups

* ToString and AsRef are in prelude, no need to import them

Auto merge of #4934 - illicitonion:exhaustive_match, r=flip1995

Update wildcard enum match lint for non_exhaustive enums

changelog: wildcard_enum_match_arm gives better suggestions for non_exhaustive enums

Auto merge of #4911 - phansch:update_changelog, r=flip1995

Update changelog for Rust 1.39 and 1.40

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

changelog: none

Auto merge of #4939 - matthiaskrgr:bump_git_dep, r=phansch

deps: bump git2 from 0.10 to 0.11

changelog: none

Auto merge of #4823 - Areredify:must_use_res, r=flip1995

Add `let_underscore_must_use` lint

changelog: closes #4812 , added a new `let_underscore_must_use` lint, moved `is_must_use_ty` to utils, added `is_must_use_fn` util function

Fix two commit hashes in changelog

Auto merge of #4941 - lzutao:utils-mod, r=phansch

a few small cleanups

changelog: none

Auto merge of #4944 - JohnTitor:rustup, r=oli-obk

Rustup to rust-lang/rust#66877

changelog: none

Rustup to rust-lang/rust#66877

update term to 0.6

a few small cleanups

implemented `let_underscore` lint

actually add files

update lints

change to pedantic

move is_must_use_ty to utils

deps: bump git2 from 0.10 to 0.11

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

Rustup to rust-lang/rust#66931

changelog: none

Rustup to rust-lang/rust#66931

Add <'_> where necessary

Remove trailling .s

Auto merge of #4937 - mikerite:fix-4824, r=phansch

Fix `map_clone` false positive

Don't lint when the item type is not a reference. `copied` only applies to references.

changelog: Fix `map_clone` false positive

Auto merge of #4930 - flip1995:unused_label, r=phansch

Deprecate unused_label lint

This lint was uplifted/turned into warn-by-default in rustc

Fixes #4925

changelog: Deprecate [`unused_label`] lint

Fix `map_clone` false positive

Don't lint when the item type is not a reference. `copied` only applies
to references.

Auto merge of #4935 - krishna-veerareddy:issue-4241-cognitive-complexity-lint-span, r=phansch

Reduce cognitive complexity lint span

Currently the cognitive complexity lint spans the entire function body making it really difficult to read and refactor the code in editors. To fix this we reduce the lint span to the function name.

changelog: Reduce cognitive complexity lint span

Fixes #4241

Update wildcard enum match lint for non_exhaustive enums

Reduce cognitive complexity lint span

Currently the cognitive complexity lint spans the entire function
body making it really difficult to read and refactor the code in
editors. To fix this we reduce the lint span to the function name.

Auto merge of #4932 - lzutao:rustup-67355, r=matthiaskrgr

rustup "Merge `ast::Mutability` and `mir::Mutability`"

cc https://github.com/rust-lang/rust/pull/67355
changelog: none

rustup "Merge `ast::Mutability` and `mir::Mutability`"

Deprecate unused_label lint

This lint was uplifted/turned into warn-by-default in rustc

Auto merge of #4922 - oli-cosmian:patch-1, r=flip1995

There are no per-file copyright headers anymore

changelog: none

Auto merge of #4929 - matthiaskrgr:rustup_25, r=matthiaskrgr

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

changelog: none

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

Auto merge of #4926 - matthiaskrgr:rurstup_24, r=matthiaskrgr

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

changelog: none

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

There are no per-file copyright headers anymore

Auto merge of #4920 - lily-commure:fix-unnecessary-filter-map-docs, r=phansch

Fix documentation example for unnecessary_filter_map.

Fixes #4919.

changelog: none

Auto merge of #4913 - mikerite:step-by-zero-20191218, r=phansch

Move `iterator_step_by_zero` and correct the documentation

Move `iterator_step_by_zero` and correct the documentation.

changelog: Corrected `iterator_step_by_zero` documentation

Update lints for `iterator_step_by_zero` changes

Auto merge of #4921 - qoh:patch-1, r=phansch

Fix 'redudant' spelling in redundant_clone docs

The word 'redundant' is spelled incorrectly in the **What it does** section of the redundant_clone lint's documentation.

changelog: Fix spelling in redundant_clone lint documentation

Fix 'redudant' spelling in redundant_clone docs

The word 'redundant' is spelled incorrectly in the **What it does** section of the redundant_clone documentation.

Fix documentation example for unnecessary_filter_map.

Auto merge of #4915 - mikerite:fix-4912-2, r=phansch

Fix `expect_fun_call` false negative on references

Closes #4912

changelog: Fix `expect_fun_call` false negative on references

Fix `expect_fun_call` false negative on references

Closes #4912

Fix `iterator_step_by_zero` description in declaration

Co-Authored-By: Phil Hansch <dev@phansch.net>

Auto merge of #4910 - krishna-veerareddy:issue-1205-cmp-nan-against-consts, r=phansch

Detect comparisons with NAN constants

Currently `cmp_nan` lint doesn't detect comparisons with NaN's if the operands are consts variables so to fix this we evaluate the const variables first before testing for NaN.

changelog: Detect comparisons with NaN constants in `cmp_nan` lint

Fixes #1205

Update changelog for Rust 1.39 and 1.40

Fix `iterator_step_by_zero` definition

Correct `iterator_step_by_zero` documentation

Update iterator_step_by_zero

Move `iterator_step_by_zero` into `methods` since it applies to all
iterators and not just ranges. Simplify the code while doing so.

Prevent `cmp_nan` when inside constants

`std::{f32,f64}::is_nan` isn't a const fn so prevent `cmp_nan`
lint from running within constant comparisons.

Detect comparisons with NAN constants

Currently `cmp_nan` lint doesn't detect comparisons with NaN's
if the operands are consts variables so to fix this we evaluate
the const variables first before testing for NaN.

Auto merge of #4907 - krishna-veerareddy:fix-clippy-build, r=phansch

Fix clippy build failure

Clippy build fails because the feature `result_map_or` has been stabilized in v1.41.0 but we still have an explicit feature attribute for it.

changelog: none

Fix clippy build failure

Clippy build fails because the feature `result_map_or` has been
stabilized in v1.41.0 but we still have an explicit feature
attribute for it.

Merge pull request #4900 from Mark-Simulacrum/niko-no-never

Update tests for revert of never type stabilization

Update tests for revert of never type stabilization

Auto merge of #4883 - krishna-veerareddy:issue-4818-cast-sign-loss-false-positive, r=flip1995

Fix false positive with cast_sign_loss lint

`cast_sign_loss` lint incorrectly suggests that the result of `checked_abs`, `rem_euclid` and `checked_rem_euclid` cannot be casted to an unsigned integer without loss.

Fixes #4818 #4764 #4743

changelog: Fix false positives in `cast_sign_loss` lint

Auto merge of #4886 - rust-lang:must-use-pub-only, r=phansch

Lint only exported must_use_candidates

As promised on #4779, here's the check for publicly visible items for `must_use_candidate`

changelog: none

Auto merge of #4889 - krishna-veerareddy:issue-3993-float-to-int-transmute, r=llogiq

Add lint to detect transmutes from float to integer

Add lint that detects transmutation from a float to an integer and suggests usage of `{f32, f64}.to_bits()` instead.

Fixes #3993

changelog: Add lint `transmute_float_to_int`

Move `transmute_float_to_int` test cases into separate file

`transmute.stderr` file line count exceeded due to the new test
cases so moving the new test cases into a separate file.

Add lint to detect transmutes from float to integer

Add lint that detects transmutation from a float to an integer
and suggests usage of `{f32, f64}.to_bits()` instead.

Lint only exported must_use_candidates

Auto merge of #4884 - RobbieClarken:warn-missing-errors, r=llogiq

Add lint for pub fns returning a `Result` without documenting errors

The Rust Book recommends that functions that return a `Result` type have [a doc comment with an `# Errors` section](https://doc.rust-lang.org/book/ch14-02-publishing-to-crates-io.html#commonly-used-sections) describing the kind of errors that can be returned. This change adds a lint to enforce this. The lint is allow by default; it can be enabled with `#![warn(clippy::missing_errors_doc)]`.

Closes #4854.

changelog: adds lint for `missing_errors_doc`

Add lint for pub fns returning a `Result` without documenting errors

The Rust Book recommends that functions that return a `Result` type have
a doc comment with an `# Errors` section describing the kind of errors
that can be returned
(https://doc.rust-lang.org/book/ch14-02-publishing-to-crates-io.html#commonly-used-sections).
This change adds a lint to enforce this. The lint is allow by default;
it can be enabled with `#![warn(clippy::missing_errors_doc)]`.

Closes #4854.

Add widening tests for `cast_sign_loss` lint

Fix false positive with cast_sign_loss lint

`cast_sign_loss` lint incorrectly suggests that the result of
`checked_abs`, `rem_euclid` and `checked_rem_euclid` cannot
be casted to an unsigned integer without loss.

Auto merge of #4880 - daxpedda:string-add, r=phansch

Fix false positive in `string_add`.

`clippy::string_add` was popping up in macros.
I'm not sure what clippy's general direction is in these matters, but I can change it to be external macros only too.

---

changelog: Fix false positives for `string_add` in macro expansions.

Change to only detect in external macros.

Fix false positive in `string_add`.

Auto merge of #4879 - matthiaskrgr:rustup_23, r=flip1995

Rustup to rust-lang/rust#66878

I need to sleep now, feel free to pick it up.
The output of the `lint_without_lint_pass` test seems to disappear, I'm not sure why.. :/

changelog: none

Rustup to rust-lang/rust#66878

Auto merge of #4877 - flip1995:manual_swap_4853, r=llogiq

Fix FP in manual_swap lint with slice-like types

Fixes #4853

changelog: Fix FP in [`manual_swap`] lint with slice-like types and make it auto applicable

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

Rustup

Included rustups:

- rust-lang/rust#66935 (syntax: Unify macro and attribute arguments in AST)
- rust-lang/rust#66941 (Remove `ord` lang item)

Fixes? #2597

changelog: none

Update op_ref.stderr

Make OP_REF lint suggestion MaybeIncorrect

cc #2597

Fire clippy::op_ref on PartialOrd but !Ord types

Rustup to rust-lang/rust#66935

Fix rustdoc examples

Make manual_swap autofixable

Formatting

Fix FP in manual_swap lint with slice-like types

Add regression test for manual_swap

Auto merge of #4875 - flip1995:rustup, r=matthiaskrgr

Rustup to rust-lang/rust#64736

cc rust-lang/rust#64736

Fixes #4872

changelog: none

Normalize custom ICE test

Fixes #4872

Rustup to rust-lang/rust#64736

Auto merge of #4873 - phansch:rustup2, r=eddyb

Rustup to https://github.com/rust-lang/rust/pull/66789

changelog: none

Use assert_crate_local for a more explicit error

`assert_crate_local` does the same as the previous `if let` but with a
more explicit error message if it's not a `ClearCrossCrate::Set`.

Rustup to https://github.com/rust-lang/rust/pull/66789

Auto merge of #4690 - lzutao:fix-test-on-non-amd64, r=flip1995

generate stderr file on 32bit pointer system

changelog: none

Auto merge of #4870 - lzutao:check-macro-missing_inline_in_public_items, r=flip1995

account for external macro in MISSING_INLINE_IN_PUBLIC_ITEMS lint

Closes #4861
changelog: Fix FP (external macro) in [`missing_inline_in_public_items`] lint

account for external macro in MISSING_INLINE_IN_PUBLIC_ITEMS lint

build(tests/fmt): use shared target dir

chore: fix and split some ui tests on 32bit system

build: set up build job for i686 targets

remove needless my_lint ui test

git quiet

Auto merge of #4866 - rust-lang:needful-doctest-main, r=flip1995

Less needless_doctest_main false positives

This checks if a) the `fn main() {}` function is empty or if the doctest contains a `static`. In both cases don't lint. While this fixes #4858 at the cost of some false negatives, but this seems a better solution than disabling the lint outright. In the long run, using `syn` should solve the issue in the right way.

changelog: none

Auto merge of #4868 - lzutao:deploy-in-dir, r=phansch

deploy: cd to out/ before adding files to git

It should work now: https://travis-ci.com/lzutao/rust-clippy/jobs/261801929
Closes #4785
changelog: none

deploy: cd to out/ before adding files to git

Auto merge of #4865 - phansch:fix_master_deployment, r=llogiq

Fix master deployment

It was failing because it could not find `.github/deploy_key.enc` in the
current directory (`./out`). Switching to the previous directory should
fix the deployment and the master build.

Example Travis log: https://travis-ci.com/rust-lang/rust-clippy/jobs/261688169#L1900

changelog: none

cc @lzutao

Less needless_doctest_main false positives

This checks if a) the `fn main() {}` function is empty or if the doctest contains a `static`. In both cases don't lint.