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

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/