Merge pull request #3285 from devonhollowood/pedantic-dogfood-items-after-statements

Pedantic dogfood: `items_after_statements`

Fix items_after_statements for `const`s

Fix items_after_statements for sub-functions

Fix items_after_statements for `use` statements

Merge pull request #3280 from d-dorazio/fix-new_without_default-should-not-fire-unsafe-new

new_without_default should not warn about unsafe new

Merge pull request #3279 from phansch/fix_fp_in_fn_to_numeric_cast_with_truncation

Fix FP in `fn_to_numeric_cast_with_truncation`

Fix FP in `fn_to_numeric_cast_with_truncation`

We only want this lint to check casts to numeric, as per the lint title.
Rust already has a built-in check for all other casts
[here][rust_check].

[rust_check]: https://github.com/rust-lang/rust/blob/5472b0718f286266ab89acdf234c3552de7e973c/src/librustc_typeck/check/cast.rs#L430-L433

new_without_default should not warn about unsafe new

Merge pull request #3273 from maccoda/master

Adding more detail to filter_map lint documentation.

Add license to README

Adding more detail to filter_map lint documentation.

Merge pull request #3269 from rust-lang-nursery/relicense

Relicense clippy to the Rust license

additional people

Add license header to other files

Add license header to Rust files

Relicense clippy

Documentation on relicensing in previous commit

Fixes #2885

Also fixes #3093, fixes #3094, fixes 3095, fixes #3096, fixes #3097, fixes #3098,
fixes #3099, fixes #3100, fixes #3230

Document relicensing process

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

Rustup for https://github.com/rust-lang/rust/pull/54741

Rustup for https://github.com/rust-lang/rust/pull/54741

Remove unused utils

Merge pull request #3251 from rust-lang-nursery/relicense-rewrite

[do not merge] Relicensing rewrite

Merge pull request #3268 from matthiaskrgr/rustup_macros_20

mini-macro: fix tests with latest rustc (rename feature: proc_macro_non_items -> proc_macro_hygiene).

mini-macro: fix tests with latest rustc (rename feature: proc_macro_non_items -> proc_macro_hygiene).

Merge pull request #3267 from matthiaskrgr/travis_osx

travis: reenable osx

travis: reenable osx

Merge pull request #3264 from joelgallant/extern-type-complexity

Fixes #3222 by checking the BareFnTy Abi type

Fix fn_to_numeric_cast UI tests

This collapses both lint tests into one file.
Somehow allowing the other lint in the respective files did not work
correctly. Maybe that's fixed as part of fixing #3198.

Merge remote-tracking branch 'origin/master' into relicense-rewrite

Solves #3222 by checking the BareFnTy Abi type

Only run tests if pointer width is 64bit

If the pointer width of the architechture is 32bit or something else,
then the tests will most likely produce different results.

Add hidden lifetime parameters to fix warning

Merge pull request #3259 from matthiaskrgr/build_warnings

resolve build warnings in clippy_lints/src/format.rs

Reimplement the `fn_to_numeric_cast_with_truncation` lint

Improve docs of fn_to_numeric_cast

Closes #2980

Merge pull request #3261 from matthiaskrgr/travis_failure

make sure travis fails when clippy does not build.

make sure travis fails when clippy does not build.

Fixes #3260

resolve build warnings in clippy_lints/src/format.rs

Merge pull request #3255 from o01eg/env_logger

Allow to debug rustc_driver via logs.

Merge pull request #3256 from rust-lang-nursery/fix-2796

Fix 2796 and 3020

Don't suggest `to_string().to_string` in USELESS_FORMAT

Allow to debug rustc_driver via logs.

Reimplement the `fn_to_numeric_cast` lint

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

Fix push_item_path call (rustup to 4cf11765dc98536c6eedf33f2df7f72f6e161263)

Fix push_item_path call (rustup to 4cf11765dc98536c6eedf33f2df7f72f6e161263)

Ignore `format!` with precision in `USELESS_FORMAT`

Add test for variable width in `USELESS_FORMAT`

Trailing newline

FIx dogfood

Reimplement the `map_clone` lint from scratch

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

relicensing: Remove map_clone

This removes the code added in https://github.com/rust-lang-nursery/rust-clippy/pull/427

relicensing: Remove fn_to_numeric_cast, fn_to_numeric_cast_with_truncation

This removes the code added in https://github.com/rust-lang-nursery/rust-clippy/pull/2814

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