Fix ty::TyKind usage

Fix TyKind::<kind> usage in codebase

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

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