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

Merge pull request #3150 from matthiaskrgr/crate_vis_feat

clippy_lints: enable crate_visibility_modifier since it is used but no longer part of 2018 edition.

add tests for #3057 and #2651

Fixes #3057
Fixes #2651

Corrected explicit_counter_loop behavior with nested loops

Fix #1219 false positive for explicit_counter_loop

clippy_lints: enable crate_visibility_modifier since it is used but no longer part of 2018 edition.

Fixes build with https://github.com/rust-lang/rust/pull/53999

Merge pull request #3146 from flip1995/fix-3145

Fix 3145

rustc_tools_util: add test

rustc_tools_util: don't hardcode crate name

Add regression test

Fix #3145 by removing assert

Updated explicit_counter_loop tests based on discussion in #3135

Merge pull request #3129 from mipli/3091-numeric-typo

Add lint for misstyped literal casting

Merge pull request #3137 from matthiaskrgr/clippy_git_version

print git commit hash and commit date in version output

Merge pull request #3139 from matthiaskrgr/toml_clippy_dev_warn

Cargo.toml: remove clippy-dev entry referencing src/main.rs as its main.rs

Merge pull request #3138 from matthiaskrgr/3126__no_warn_multi_newline

print_with_newline / write_with_newline: don't warn about string with several `\n`s in them.

Simplified boolean expression for checking literal suffixes

Reduce number of split_at calls

Corrected explicit_counter_loop missing lints if variable used after loop

print git commit hash and commit date in version output

clippy 0.0.212 (964fcbe0 2018-09-06)

Cargo.toml: remove clippy-dev entry referencing src/main.rs as its main.rs.

Resolves warning:
warning: file found to be present in multiple build targets: ./src/main.rs

print_with_newline / write_with_newline: don't warn about string with several `\n`s in them.

Fixes #3126

rustfmt clippy_lints/src/write.rs

Merge pull request #3136 from mikerite/driver-rustc-arg-2

driver: Improve check for rustc arg

Merge pull request #2985 from phansch/riir_update_lints

update_lints rewrite: Add structure and --print-only

Pass by ref instead of value

More refactoring

Merge pull request #3133 from matthiaskrgr/link

remove "clippy::" lint prefix from lint name in doc url.

driver: Improve check for rustc arg

The rustc arg might not be exactly "rustc". It may be any path to a rustc
executable (especially if the RUSTC environment variable is set when
executing cargo). Rather check that it is a path with 'rustc' file stem.

Closes #1219 false positive for explicit_counter_loop

remove "clippy::" lint prefix from lint name in doc url.

Fixes #3132

Add lint for misstyped literal casting

Merge pull request #3131 from JoshMcguigan/ptr_arg-2293

Fix #2293 - ptr_arg

Added test case for ptr_arg

Remove duplicated .gitignore

Return impl Iterator instead of Vec<Lint>

This makes the API of `lib.rs` a bit more flexible.

Merge pull request #3108 from daubaris/range-plus-one

range_plus_one suggestion should not remove braces fix

switched to ticks for chars

Merge branch 'master' of https://github.com/rust-lang-nursery/rust-clippy into range-plus-one

Merge pull request #3125 from nrc/pub-conf

Make `Default` do what `default` used to do

Merge pull request #3124 from phansch/cleanup_old_min_version_stuff

Cleanup old min_version stuff

Merge pull request #3109 from shssoichiro/3034-needless-collect

Lint against needless uses of `collect()`

Delete needless file

More if_chain

Rustup

Apply applicability

Majority of PR changes

Lint against needless uses of `collect()`

Handles cases of `.collect().len()`, `.collect().is_empty()`, and
`.collect().contains()`. This lint is intended to be generic enough to
be added to at a later time with other similar patterns that could be
optimized.

Closes #3034

Make `Default` do what `default` used to do

Cleanup old min_version stuff

This cleans up a few leftover things after
https://github.com/rust-lang-nursery/rust-clippy/pull/3018

Run clippy on clippy_dev, too

Refactor to use into_group_map from Itertools

s/direntry/dir_entry

Use insignificant whitespace mode for nice regex

cargo update in clippy_dev

update_lints rewrite: Add structure and --print-only

fixed known problems expression

Merge branch 'master' of https://github.com/rust-lang-nursery/rust-clippy into range-plus-one

backticks and testcase

Merge pull request #3092 from illicitonion/issue-2879

default_trait_access skips <F as Default>::default()

Merge branch 'master' of https://github.com/rust-lang-nursery/rust-clippy into range-plus-one