Merge pull request #3245 from JoshMcguigan/wrong_self_convention-1530

Correct false positive in wrong_self_convention lint for to_mut

Correct false positive in wrong_self_convention lint for to_mut

Merge pull request #3243 from flip1995/appveyor_disable_dogfood

Disable dogfood under windows until rust-lang-nursery/rustup.rs#1499 is merged

Merge pull request #3249 from rust-lang-nursery/rustup

ScalarMaybeUndef -> Scalar (Rustup to e812ca472a2a5284e9f15cd9af32285d7ff3fd39)

Merge pull request #3250 from rust-lang-nursery/levels-readme

Mention -A and -W in readme

Mention -A and -W in readme

ScalarMaybeUndef -> Scalar (Rustup to e812ca472a2a5284e9f15cd9af32285d7ff3fd39)

Merge pull request #3247 from rust-lang-nursery/dummy

Add dummy clippy crate for publishing

publish = false

Add dummy clippy crate for publishing

Merge pull request #3248 from phansch/fix_ice_2831

Fix 'impossible case reached' ICE

Fix 'impossible case reached' ICE

Disable dogfood until rust-lang-nursery/rustup.rs#1499 is merged

Merge pull request #3242 from matthiaskrgr/rm_cargo_ed_feat

remove cargo edition 2018 feature gate.

Merge pull request #3241 from phansch/remove_clippy_service

Remove clippy-service update from CI

Remove clippy-service token

remove cargo edition 2018 feature gate.

Rust and the cargo used to bootstrap was updated in https://github.com/rust-lang/rust/pull/54601 which now has the 2018 edition stabilized.

Merge pull request #3240 from matthiaskrgr/travis_3239

travis: sleep after putting out logs (try to fix truncated logs)

Remove clippy-service update from CI

clippy-service hasn't been working for a long time now.

travis: sleep after putting out logs (try to fix truncated logs)

Merge pull request #3223 from mikerite/unnecessary_filter_map

Implement unnecesary_filter_map lint

Merge branch 'master' into unnecessary_filter_map

Fix update_lints.py for dir modules

Merge pull request #3235 from matthiaskrgr/rustc_tools__beta_fix

rustc_tools_util: don't require test feature.

rustc_tools_util: remove test and tool_lints features, both are actually unused.

Fixes build with beta.

Merge pull request #3238 from JoshMcguigan/excessive_precision-3180

Fixes #3180, suppress excessive_precision lint for floats with no decimal part

Fixes #3180, suppress excessive_precision lint for floats with no decimal part

Merge branch 'master' into unnecessary_filter_map

rustfmt

Move unnecessary_filter_map to a submodule

Move tests into separate file

Merge pull request #3224 from matthiaskrgr/clippy_self__use_self

fix some clippy::pedantic warnings found in the codebase.

Merge pull request #3178 from ms2300/bad_unwrap

Fix for bad get unwrap suggestion

Merge pull request #3227 from matthiaskrgr/cargo_lock

consistently gitignore all Cargo.lock files

consistently gitignore all Cargo.lock files

Fix warnings in clippy_lints

Merge branch 'master' into unnecessary_filter_map

fix clippy::single-match-else and clippy::match_same_arms warnings in clippys codebase

fix all clippy::use_self pedantic warnings found in the codebase.

cc #3172

Merge pull request #3226 from matthiaskrgr/rustup_smallvec

rustup

Merge pull request #3225 from matthiaskrgr/dogfood

tests: dogfood: extend to run with --all-features and clippy::internal enabled.

rustup

fix breakage by https://github.com/rust-lang/rust/pull/53824

use smallvec crate instead of rustcs type alias.

tests: dogfood: extend to run with --all-features and clippy::internal enabled.
Run it on rustc_tools_util and clippy_dev as well.

Implement unnecesary_filter_map lint

Merge pull request #3218 from Hanaasagi/fix-travis-yaml-demo

fix invalid travis-yaml in README

fix invalid travis-yaml in README

Merge pull request #3203 from yaahallo/master

Lint for chaining flatten after map

Merge pull request #3212 from matthiaskrgr/clippy_dev_ed2018

clippy_dev: port to edition 2018

Lint for chaining flatten after map

This change adds a lint to check for instances of `map(..).flatten()`
that can be trivially shortened to `flat_map(..)`

Closes #3196

Merge pull request #3213 from mikerite/fix-3204

Fix single_char_pattern crash (#3204)

Fix single_char_pattern crash (#3204)

This commit fixes the crash by removing constant checking from the lint.

Closes #3204.

Full fix of get unwrap issue

#3006 : Fixing for .get().unwrap().foo()

Merge pull request #3207 from mikerite/fix-3206

Fix double_parens false positive

Merge pull request #3191 from vi/suggest_with_applicability

Use span_suggestion_with_applicability instead of span_suggestion

clippy_dev: port to edition 2018

Merge pull request #3195 from JayKickliter/jsk/mem_replace_opt_w_none

Add lint for `mem::replace(.., None)`.

Mention `rustup self update` (fixes #3211)

Fix double_parens false positive

Closes #3206

Another Applicability adjustment

Applicability adjustment per additional comments

Fill in Applicability from review comments by @flip1995

Fix indents

Apply subset of "cargo fmt".

Add forgotten function: span_suggestion*s* to the previous refactoting

Supplement DiagnosticBuilderExt with Applicability

Use span_suggestion_with_applicability instead of span_suggestion

Merge pull request #3201 from phansch/rust_master_in_contributing_docs

Mention how to install master Rust in CONTRIBUTING

Mention rustup component history

s/rustc/Rust/

s/rust/Rust, repeat 'Clippy' less

mem_replace: fix grammar.

mem_replace: match on path.

mem_replace: apply update_lints tool.

mem_replace: make examples compilable.

Add lint for `mem::replace(.., None)`.

Suggest `Option::take()` as an alternative.

Mention how to install master Rust in CONTRIBUTING

Merge pull request #3197 from mikerite/travis_lint_list

Make travis check lint list

Merge pull request #3187 from flip1995/internal_fn

New internal lint: compiler_lint_functions

Update lint list

Make travis check lint list

Merge pull request #3194 from flip1995/c_void

Fix c_void path

Fix c_void path

This got changed in rust-lang/rust#53910

Merge pull request #3192 from mikerite/refactor_20180916

Remove unneeded check for method call

Remove unneeded check for method call

The check can be removed because the call to `method_chains_args`
already performs this check.

Merge pull request #3190 from eddyb/rextern-2

Reintroduce `extern crate` for non-Cargo dependencies, in tests.

Reintroduce `extern crate` for non-Cargo dependencies, in tests.

Merge pull request #3068 from matthiaskrgr/string

convert "".to_string() and "".to_owned() to String::new()

Replace another occurrence of "".to_owned()

convert "".to_string() and "".to_owned() to String::new()

Formatting

Fix warnings of compiler_lint_functions

Add internal lint compiler_lint_functions

Merge pull request #3189 from eddyb/rextern

Reintroduce `extern crate` for non-Cargo dependencies.

Revert "the cargo feature: edition 2018 is stabilized in current nightly"

This reverts commit 404a09d61cf7bc8e5d4e92ae9052a4e10d2ff07d.

Fix `useless_attribute` to also whitelist `unused_extern_crates`.

Reintroduce `extern crate` for non-Cargo dependencies.

Merge pull request #3186 from flip1995/pedantic_filter_map

Fix pedantic filter_map warnings

Reapply: "the cargo feature: edition 2018 is stabilized in current nightly"

This reapplies commit 404a09d.

Cargo got bumped in rust-lang/rust#54210

Fix pedantic filter_map warnings