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

constants: add u128 i128 builtin types and fix outdated url

Update lints

Lint only the first statment/expression after alloc

Instead of searching for all the successive expressions after a vector
allocation, check only the first expression.

This is done to minimize the amount of false positives of the lint.

Fix some warnings related to Self

Rename some symbols

Renamed some symbols in order to make them a little bit more accurate.

Split lint into slow and unsafe vector initalization

Add unsafe set_len initialization

Add slow zero-filled vector initialization lint

Add lint to detect slow zero-filled vector initialization. It detects
when a vector is zero-filled with extended with `repeat(0).take(len)`
or `resize(len, 0)`.
This zero-fillings are usually slower than simply using `vec![0; len]`.

Merge pull request #3439 from dtolnay/npbv

Downgrade needless_pass_by_value to allow by default

Merge pull request #3453 from phansch/remove_sudo_false

Travis: Remove `sudo: false`

Travis: Remove `sudo: false`

Merge pull request #3452 from matthiaskrgr/rustup

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

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

Merge pull request #3449 from matthiaskrgr/dep

dependencies: update pulldown-cmark from 0.1 to 0.2

dependencies: update pulldown-cmark from 0.1 to 0.2

Merge pull request #3447 from phansch/small_rename

s/file_map/source_map

Merge pull request #3442 from rust-lang/version_ids

Enable rustup clippy to refer to the correct documentation

Merge branch 'master' into version_ids

Merge pull request #3446 from matthiaskrgr/url_update

update urls (rust-lang-nursery => rust-lang)

s/file_map/source_map

missed another one in the README

run "util/dev update_lints"

rust-lang-nursery/rust-clippy => rust-lang/rust-clippy

Merge #3437

3437: issue #3318 Run trivially_copy_pass_by_ref for traits r=flip1995 a=waynr

Addresses #3318

Co-authored-by: Wayne Warren <wayne.warren.s@gmail.com>
Co-authored-by: Philipp Krones <hello@philkrones.com>

Merge #3443

3443: Document how to lint local Clippy changes with locally built Clippy r=flip1995 a=waynr

Co-authored-by: Wayne Warren <wayne.warren.s@gmail.com>

Address 'clippy::single-match' dogfood lint

Fix nit

Co-Authored-By: waynr <wayne.warren.s@gmail.com>

Address travis CI lint failure

Update trivially_copy_pass_by_ref with Trait stderr output

issue#3318 run trivially_copy_pass_by_ref for traits

Update trivially_copy_pass_by_ref with Trait examples

Fix awkward wording

Document how to lint local Clippy changes with locally built Clippy

Merge #3441

3441: rustup https://github.com/rust-lang/rust/pull/52591 r=flip1995 a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Enable rustup clippy to refer to the correct documentation

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

Downgrade needless_pass_by_value to allow by default

I noticed that I suppress this lint in many of my projects.

https://github.com/search?q=needless_pass_by_value+user%3Adtolnay&type=Code
https://github.com/search?q=needless_pass_by_value+user%3Aserde-rs&type=Code

Upon further inspection, this lint has a *long* history of false
positives (and several remaining).

Generally I feel that this lint is the definition of pedantic and should
not be linted by default.

    #[derive(Debug)]
    enum How {
        ThisWay,
        ThatWay,
    }

    // Are we really better off forcing the call sites to write f(&_)...?
    fn f(how: How) {
        println!("You want to do it {:?}", how);
    }

    fn main() {
        f(How::ThatWay);
    }

Merge #3432

3432: match_ref_pats: don't emit suggestions inside of a macro r=Manishearth a=flip1995

Fixes #2636

Co-authored-by: flip1995 <hello@philkrones.com>

Merge #3435

3435: remove unused allow() attributes, NFC r=flip1995 a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

remove unused allow() attributes, NFC

Merge #3431

3431: rustup https://github.com/rust-lang/rust/pull/55852/ r=flip1995 a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Add regression test

Don't emit suggestion when inside of a macro

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

Merge #3419

3419: Linter to check that all common metadata is included r=flip1995 a=ysimonson

Addresses https://github.com/rust-lang-nursery/rust-clippy/issues/1793

This is not ready to be merged yet. First the associated changes to `cargo_metadata` need to be merged: https://github.com/oli-obk/cargo_metadata/pull/55

Co-authored-by: Yusuf Simonson <simonson@gmail.com>

Merge #3428

3428: Typo "too" -> "foo" in format.rs r=matthiaskrgr a=ama0

Co-authored-by: Anton <ama0@users.noreply.github.com>

Fix "too" -> "foo" typo in format.rs

Merge #3427

3427: Fix wrong suggestion for `redundant_closure_call` r=oli-obk a=mikerite

Fixes #1684

Co-authored-by: Michael Wright <mikerite@lavabit.com>

Fix `use_self` violation

Fix wrong suggestion for `redundant_closure_call`

Fixes #1684

Check for common metadata

Merge #3426

3426: Fix `use_self` false positive on `use` statements r=flip1995 a=mikerite

Fixes #3425

Co-authored-by: Michael Wright <mikerite@lavabit.com>

Fix `use_self` false positive on `use` statements

Merge #3423

3423: Fix `use_self` false positive r=flip1995 a=mikerite

This fixes the first error reported in issue #3410.

Co-authored-by: Michael Wright <mikerite@lavabit.com>

Fix `use_self` false positive

This fixes the first error reported in issue #3410.

Merge #3407

3407: Fix `possible_missing_comma` false positives r=oli-obk a=mikerite

`possible_missing_comma` should only trigger when the binary operator has
unary equivalent. Otherwise, it's not possible to insert a comma without
breaking compilation. The operators identified were `+`, `&`, `*` and `-`.

This fixes the specific examples given in issues #3244 and #3396
but doesn't address the conflict this lint has with the style of starting
a line with a binary operator.

Co-authored-by: Michael Wright <mikerite@lavabit.com>

Remove `+` from `has_unary_equivalent`

Rust doesn't has a unary + operator!

Merge branch 'master' into fix-missing-comma-fp

Merge #3408

3408: RIIR update lints: Add check mode (update_lints.py rewrite complete) r=oli-obk a=phansch

This finishes up the rewrite of `update_lints.py` in Rust. More
specifically, this

* adds the `--check` flag and handling to clippy_dev
* tracks file changes over the different calls to `replace_region_in_file`
* only writes changes to files if the `--check` flag is *not* used
* runs `./util/dev update_lints --check` on CI instead of the old script
* replaces usage of the `update_lints.py` script with an error

`./util/dev update_lints` behaves 99% the same as the python script.
The only difference that I'm aware of is an ordering change to
`clippy_lints/src/lib.rs` because underscores seem to be sorted
differently in Rust and in Python.

:checkered_flag:

cc #2882

Co-authored-by: Philipp Hansch <dev@phansch.net>
Co-authored-by: Philipp Krones <hello@philkrones.com>

Fix dogfood

Update println! formatting

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

Fix false positive in check mode caused by `gen_deprecated`

`gen_deprecated` should never have created the string with linebreaks.

Using a single string broke the check because the changed lines were
different.

RIIR update lints: Add check mode (update_lints.py rewrite complete)

This finishes up the rewrite of `update_lints.py` in Rust. More
specifically, this

* adds the `--check` flag and handling to clippy_dev
* tracks file changes over the different calls to `replace_region_in_file`
* only writes changes to files if the `--check` flag is *not* used
* runs `./util/dev update_lints --check` on CI instead of the old script
* replaces usage of the `update_lints.py` script with an error

`./util/dev update_lints` behaves 99% the same as the python script.
The only difference that I'm aware of is an ordering change to
`clippy_lints/src/lib.rs` because underscores seem to be sorted
differently in Rust and in Python.

:checkered_flag:

Merge #3353

3353: float support added for mistyped_literal_suffixes lint r=mikerite a=Maxgy

I implemented the mistyped_literal_suffixes lint for float literals.

```
#![allow(unused_variables)]

fn main() {
    let x = 1E2_32;
    let x = 75.22_64;
}
```
Given the above, the additional check suggests the variables to be written as `let x = 1E2_f32` and `let x = 75.22_f64`.

Fixes #3167

Co-authored-by: Maxwell Anderson <maxwell.brayden.anderson@gmail.com>

Merge #3411

3411: rustup https://github.com/rust-lang/rust/pull/55665 (pass contexts by reference) r=matthiaskrgr a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

rustup https://github.com/rust-lang/rust/pull/55665 (pass contexts by reference)

changed into_iter to iter and fixed a lint check

Fix `collapsible_if` error

Fix `possible_missing_comma` false positives

`possible_missing_comma` should only trigger when the binary operator has
unary equivalent. Otherwise, it's not possible to insert a comma without
breaking compilation. The operators identified were `+`, `&`, `*` and `-`.

This fixes the specific examples given in issues #3244 and #3396
but doesn't address the conflict this lint has with the style of starting
a line with a binary operator.

Merge #3399

3399: RIIR update lints: Generate modules section and lint group sections r=flip1995 a=phansch

This adds the last missing parts of the generating code.

cc #2882

Co-authored-by: Philipp Hansch <dev@phansch.net>

Merge pull request #3404 from matthiaskrgr/rustup

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

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

Fix typo

Improve clippy_dev help text

RIIR update lints: Generate lint group registrations

Test clippy_dev on CI and fix test

Merge #3400

3400: Fix a false-positive of needless_borrow r=phansch a=sinkuu

Co-authored-by: Shotaro Yamada <sinkuu@sinkuu.xyz>

Merge #3123

3123: Lint #[cfg_attr(rustfmt, rustfmt_skip)] and suggest #[rustfmt::skip] r=phansch a=flip1995

Closes #3121

Co-authored-by: flip1995 <9744647+flip1995@users.noreply.github.com>
Co-authored-by: flip1995 <hello@philkrones.com>

RIIR update lints: Generate modules section

Update stderr

Rename test files

Also lint cfg_attr(.., rustfmt::skip)

Add tests from rustfmt::skip test file

Run update_lints.py script

Add test for non-crate-level inner attributes

Differ between inner and outer attributes

Add tests

Add cfg_attr(rustfmt) lint

Merge #3344

3344: Added lints `into_iter_on_ref` and `into_iter_on_array`. r=phansch a=kennytm

Fixes #1565.

`into_iter_on_array` is a correctness lint against `into_iter()` on `[T; n]` and `PathBuf` (please provide a concise noun that covers both types 🙃).

`into_iter_on_ref` is a style lint against `into_iter()` on `&C` where `C` is `[T]`, `Vec<T>`, `BTreeSet<T>` etc.

Both suggests replacing the `into_iter()` with `iter()` or `iter_mut()`.

`into_iter_on_array` is a correctness lint since it is very likely the standard library would provide an `into_iter()` method for `[T; n]` (rust-lang/rust#25725) and existing type inference of `[a, b, c].into_iter()` will be broken. `PathBuf` is also placed under this lint since `PathBuf::into_iter` currently doesn't exist and it makes some sense to implement `IntoIterator` on it yielding `OsString`s.

Co-authored-by: kennytm <kennytm@gmail.com>

Addressed comments.

Fix dogfood error.

Added lints `into_iter_on_ref` and `into_iter_on_array`. Fix #1565.

Merge #3161

3161: New lint: Unknown clippy lints r=phansch a=flip1995

This is the Clippy version of the `rustc` lint `unknown_lints`. The behavior of this lint is pretty much the same.

Before this is merged a small change in the compiler needs to be done: `CheckLintNameResult` needs to be public. See rust-lang/rust#54106

Co-authored-by: flip1995 <9744647+flip1995@users.noreply.github.com>
Co-authored-by: flip1995 <hello@philkrones.com>

Allow single_match_else

Update stderr