Move unnecessary_filter_map to a submodule

Move tests into separate file

Fix warnings in clippy_lints

Merge branch 'master' into unnecessary_filter_map

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.

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

Merge pull request #3185 from matthiaskrgr/readme_travis

readme: show how to make the travis job fail when using clippy

readme: show how to make the travis job fail when using clippy

Merge pull request #3182 from phansch/explain_travis

Explain Travis CI usage

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

Add match on StructCtor

Add match on StructCtor

Explain Travis CI usage

Closes #2432

Merge pull request #3140 from matthiaskrgr/redundant_casts

fix warnings about trivial casts, mostly {i,u}128 -> {i,u}128, such as "i128::min_value() as i128"

Merge pull request #3174 from matthiaskrgr/RTU_clippy_cleanup

rustc_tools_util: clean up pedantic clippy warnings

Merge pull request #3173 from phansch/update_issue_template

Update ISSUE_TEMPLATE

Merge pull request #3165 from phansch/cleanup_readme_for_clippy_preview_on_stable

Cleanup README for clippy-preview on stable

rustc_tools_util: clean up pedantic clippy warnings

Update ISSUE_TEMPLATE

Remove rust-update script

We no longer need this as clippy is installed through rustup now.

Cleanup README for clippy-preview on stable

With the 1.29 release, the `clippy-preview` component will be available
on stable which means we don't need nightly/beta anymore.

Merge pull request #3166 from flip1995/travis_internal

Run internal lints on the Clippy code base

Merge pull request #3171 from rust-lang-nursery/README-tool-lints

README: More detailed explanation of tool_lints

README: More detailed explanation of tool_lints

cc #3164

fix warnings about trivial casts, mostly {i,u}128 -> {i,u}128, such as "i128::min_value() as i128"

Merge pull request #3085 from mikerite/revert-98dbce

Revert "Fix E0502 warnings"

Merge pull request #3169 from yangby-cryptape/fix-typo-in-examples

Fix typo in examples.

Remove a wrong suggestion.

Fix typo in examples.

Change Hash{Map, Set} to FxHash{Map, Set}

Fix lint_without_lint_pass lint

Also run internal lints on the code base

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

This reverts commit 404a09d61cf7bc8e5d4e92ae9052a4e10d2ff07d.

Don't use the old  feature gate

Merge pull request #3160 from matthiaskrgr/ed_2018

the cargo feature: edition 2018 is stabilized in current nightly

Merge pull request #3152 from PSeitz/master

fixes #3151 by skipping the lint instead of crashing

add spaces

Merge pull request #3157 from matthiaskrgr/Vinfo_impl_debug

impl std::fmt::Debug for VersionInfo

the cargo feature: edition 2018 is stabilized in current nightly

Merge pull request #3135 from JoshMcguigan/explicit_counter_loop-1219

Closes #1219 false positive for explicit_counter_loop

impl std::fmt::Debug for VersionInfo

For clippy, this would print:
VersionInfo { crate_name: "clippy", major: 0, minor: 0, patch: 212, commit_hash: "084be7ba", commit_date: "2018-09-09" }

Merge pull request #3148 from matthiaskrgr/rustc_tools_util_tests

rustc_tools_util: add tests

Merge pull request #3149 from matthiaskrgr/test_3075

add tests for #3057.

fixes #3151 by skipping the lint instead of crashing