Only check the assignment found at last

If there are more than one such assignment, the last one may be
the one supplied to `clone` method.
Makes `find_stmt_assigns_to` internally reverses the iterator to make
the intent to "iterate statements backward" clear.

Add comment and rename

Apply redundant_clone on clippy

Lint redundant clone of projection

Fix test

`if true` is recognized by MIR optimization.

Cleanup

Merge pull request #3512 from matthiaskrgr/rustup

rustup https://github.com/rust-lang/rust/pull/56502 ( .hir -> .hir() )

s/rustfmt-preview/rustfmt/

Swap if branches

Fix c_void false positive caused by libc refactoring

The path of `libc::c_void` has changes in https://github.com/rust-lang/libc/commit/5c1a6b8a6d558882927a0816d91c01b9c2a88018
The DefId path is now always platform specific like
`libc::windows::c_void`. This fixes our c_void detection to only check
the first and last elements.

rustup https://github.com/rust-lang/rust/pull/56502 ( .hir -> .hir() )

Merge pull request #3336 from HMPerson1/clone_on_copy_deref

Fix `clone_on_copy` not detecting derefs sometimes

Merge pull request #3507 from flying-sheep/patch-1

typo: emum → enum

typo: emum → enum

Merge pull request #3257 from o01eg/remove-sysroot

Don't try to determine sysroot. rustc_driver will use default value.

Merge pull request #3501 from matthiaskrgr/rustup

rustup clippy build with latest rustc

Merge pull request #3502 from rust-lang/nrc-patch-1

Remove -preview suffix from README

Remove -preview suffix from README

rustup clippy build with latest rustc
(breakage due to https://github.com/rust-lang/rust/commit/08f8faedd0e30f45762afbb8d4873f7041e7462c )

Fixes #3500

Merge pull request #3497 from daxpedda/master

Fix bug in `implicit_return`.

Forgot to remove some debugging code ...

Improved code noted by clippy.

Merge pull request #3496 from fkohlgrueber/macro-in-closure-in-if-cond

fix #3482 and add ui test for it

Fix bug in `implicit_return`.
Bug was already covered by test, but test was not checked for.

Fix format.

Add sysroot gettinh code to dogfood tests.

fix #3482 and add ui test for it

Add sysroot getting code to tests.

Don't try to determine sysroot. rustc_driver will use default value.

Merge pull request #3494 from daxpedda/master

Added `IMPLICIT_RETURN` lint.

Merge pull request #3444 from waynr/fix-dogfood-tests

Fix dogfood tests

Don't change current working directory of cargo tests

Use cargo's "PROFILE" envvar and set CLIPPY_DOGFOOD

Use dogfood_runner for deterministic test ordering

Remove unnecessary documentation

Fix dogfood tests.

Added additional reasoning to `Why is this bad?`.
Added comment to explain usage of MIR.

Renamed to `implicit_return`.
Covered all other kinds besides `ExprKind::Lit`.
Added check for replacing `break` with `return`.

Merge pull request #3495 from flip1995/tykind_fix

Fix usage of ty::TyKind

Fix ty::TyKind usage

Fix TyKind::<kind> usage in codebase

Appeasing the Test Gods.
Seems I'm not smart enough to run the tests locally before committing.

Renamed `forced_return` to `missing_returns`.
Better clarification in the docs.
Ran `update_lints`.

Added `FORCED_RETURN` lint.

Merge pull request #3490 from phansch/extract_single_match_else_ui_test

Extract single_match_else UI test

Merge pull request #3473 from lucasloisp/additional-bool-comparisons

Adds inequality cases to bool comparison (#3438)

Merge pull request #3488 from phansch/fix_cargo_lint_bug

Fix wildcard_dependencies false positive

Extract single_match_else UI test

There's only one test currently.
I also updated the lint doc with a 'good' example and changed the lint
help text a bit.

cc #2038

cargo fmt

Adds inequality cases to bool comparison lint

The lint now checks cases like `y != true`

Merge pull request #3478 from dtolnay/setlen

Remove unsafe_vector_initialization lint

Remove unsafe_vector_initialization lint

Merge pull request #3479 from dtolnay/literal

Downgrade large_digit_groups to pedantic

Fix wildcard_dependencies false positive

This now only checks for wildcard_dependencies if the source is a
non-git source.

I tried adding a compiletest suite for the cargo lints, but I was unable
to override the `Cargo.toml` of the original executable.

I tested this manually by modifying the main `Cargo.toml`.

Fixes #3458

Merge pull request #3487 from phansch/fix_fmt

Fix rustfmt format

Fix rustfmt format

Merge pull request #3486 from phansch/make_windows_travis_run_more_stuff

Travis: Add rustc sysroot bin to PATH for windows build

Disable incremental build for windows

Testing if this speeds up compilation time for the Windows CI build.

Travis: Add rustc sysroot bin to PATH for windows build

Merge pull request #3483 from phansch/mention_triage_procedure_in_contributing

Mention triage procedure in contributing.md

Merge pull request #3485 from phansch/fix_some_nursery_links

Fix some nursery links

Fix some nursery links

Merge pull request #3484 from phansch/fix_windows_lineendings

git: Enforce LF lineendings for everything

Enforce LF lineendings for everything

Someone on discord reported issues with UI tests.

This should make sure that git never automatically converts lineendings
for text files to `CRLF`. They should always be `LF` now.

Probably this means that we can stop using dos2unix for #3306, too.

Taken from [Rust's .gitattributes file](https://github.com/rust-lang/rust/blob/master/.gitattributes).

Mention triage procedure in contributing.md

Keep testing large_digit_groups as ui test

Keep testing unsafe_vector_initialization as ui test

Downgrade large_digit_groups to pedantic

I believe if the user already decided to put underscores in their
literal, Clippy should be willing to believe that they put a number of
underscores that they felt was readable.

Downgrade unsafe_vector_initialization to restriction

This lint looks for:

    let mut vec = Vec::with_capacity(len);
    vec.set_len(len);

The suggested replacement is `vec![0; len]`.

This is far too opinionated to be a deny-by-default lint because the performance
characteristics of the suggested replacement are totally different.

I am not convinced that this lint has value beyond what deny(unsafe_code) gives
you. Unsafe code is unsafe but please don't deny-by-default lint it if that's
the only reason.

Merge pull request #3471 from matthiaskrgr/mamor_warn

remove macro_at_most_once_rep feature attribute since it's stable

remove macro_at_most_once_rep feature attribute since it's stable

Warning was:
warning: the feature `macro_at_most_once_rep` has been stable since 1.32.0 and no longer requires an attribute to enable
  --> clippy_lints/src/lib.rs:19:12
   |
19 | #![feature(macro_at_most_once_rep)]
   |            ^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: #[warn(stable_features)] on by default

Merge pull request #3465 from flip1995/rustfmt

rustfmt everything once and for all

Merge pull request #3466 from phansch/clippy_rfc_readme

Update docs in regards to the merged RFC

Add missing word

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

Update docs in regards to the merged RFC

Merge pull request #3464 from flip1995/no_fork_integration

Don't run integration tests in forks

Document how to run rustfmt in CONTRIBUTING.md

Don't run integration tests in forks

Fix some formatting issues

Run rustfmt on clippy_lints

Run rustfmt on build.rs

Run rustfmt on clippy_dev

Run rustfmt on rustc_tools_util

Run rustfmt on src

Run rustfmt on the tests

Let travis run cargo fmt --all -- --check

Error on line overflow

Merge pull request #3459 from flip1995/sugg_appl

Add Applicability to suggestion lints: Take 2

Fix NAIVE_BYTECOUNT applicability

Fix dogfood error

Change Applicability of MISTYPED_LITERAL_SUFFIX

Add applicability level to (nearly) every span_lint_and_sugg function

Update stderr file

Fix bugs and improve documentation

Some bugs and some documentation is unrelated to the Applicability change, but
these bugs were serious and the documentation was kind of required to
understand what's going on.

Add Applicability::Unspecified to span_lint_and_sugg functions

Introduce snippet_with_applicability and hir_with_applicability functions

Merge pull request #3412 from matthiaskrgr/readme_2

readme: tell how to install clippy on travis from git if it is not shipped with a nightly.

readme: tell how to install clippy on travis from git if it is not shipped with a nightly.

Merge pull request #3365 from gnieto/lint/slow-initialization

Add slow vector initializations lint

Merge pull request #3456 from matthiaskrgr/builtin

constants: add u128 i128 builtin types and fix outdated url