Run update_lints.py script

Add test for non-crate-level inner attributes

Differ between inner and outer attributes

Add tests

Add cfg_attr(rustfmt) lint

Merge #3344

3344: Added lints `into_iter_on_ref` and `into_iter_on_array`. r=phansch a=kennytm

Fixes #1565.

`into_iter_on_array` is a correctness lint against `into_iter()` on `[T; n]` and `PathBuf` (please provide a concise noun that covers both types 🙃).

`into_iter_on_ref` is a style lint against `into_iter()` on `&C` where `C` is `[T]`, `Vec<T>`, `BTreeSet<T>` etc.

Both suggests replacing the `into_iter()` with `iter()` or `iter_mut()`.

`into_iter_on_array` is a correctness lint since it is very likely the standard library would provide an `into_iter()` method for `[T; n]` (rust-lang/rust#25725) and existing type inference of `[a, b, c].into_iter()` will be broken. `PathBuf` is also placed under this lint since `PathBuf::into_iter` currently doesn't exist and it makes some sense to implement `IntoIterator` on it yielding `OsString`s.

Co-authored-by: kennytm <kennytm@gmail.com>

Addressed comments.

Fix dogfood error.

Added lints `into_iter_on_ref` and `into_iter_on_array`. Fix #1565.

Merge #3161

3161: New lint: Unknown clippy lints r=phansch a=flip1995

This is the Clippy version of the `rustc` lint `unknown_lints`. The behavior of this lint is pretty much the same.

Before this is merged a small change in the compiler needs to be done: `CheckLintNameResult` needs to be public. See rust-lang/rust#54106

Co-authored-by: flip1995 <9744647+flip1995@users.noreply.github.com>
Co-authored-by: flip1995 <hello@philkrones.com>

Allow single_match_else

Update stderr

Add copyright statement©

Fix typos

Fix dogfood error

Fix typo and indentation

run update_lints script

Add tests for unknwon_clippy_lints lint

Add new lint: unknwon_clippy_lintsg

Merge #3402

3402: clippy: fix pedantic warnings and run clippy::pedantic lints on the codebase r=oli-obk a=matthiaskrgr

Turn on pedantic lints in dogfood and base tests.

needless_bool: fix clippy::items-after-statements
redundant_pattern_matching: fix clippy::similar-names
mods.rs: fix clippy::explicit-iter-loop
returns.rs: allow clippy::cast-possible-wrap

Fixes #3172

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

clippy: fix pedantic warnings and run clippy::pedantic lints on the codebase.

Turn on pedantic lints in dogfood and base tests.

needless_bool: fix clippy::items-after-statements
redundant_pattern_matching: fix clippy::similar-names
mods.rs: fix clippy::explicit-iter-loop
returns.rs: allow clippy::cast-possible-wrap

Fixes #3172

Merge #3387

3387: Replace big if/else expression with match r=flip1995 a=mikerite

Co-authored-by: Michael Wright <mikerite@lavabit.com>

Merge #3397 #3398

3397: UI test cleanup: Extract expect_fun_call tests r=matthiaskrgr a=phansch

Note that the new stderr file does not include a `shadow-unrelated`
error, because the new UI test file does not use `#![warn(clippy::all)]`

cc #2038

3398: UI test cleanup: Extract match_overlapping_arm tests r=matthiaskrgr a=phansch

cc #2038

Co-authored-by: Philipp Hansch <dev@phansch.net>

Merge #3388

3388: RIIR update lints: Generate deprecated lints r=phansch a=phansch

The update script now also generates the 'register_removed' section in
`clippy_lints/src/lib.rs`.

Also, instead of using `let mut store ...`, I added a new identifier
line so that the replacement will continue to work in case `let mut
store ...` ever changes.

cc #2882

Co-authored-by: Philipp Hansch <dev@phansch.net>

UI test cleanup: Extract match_overlapping_arm tests

UI test cleanup: Extract expect_fun_call tests

Note that the new stderr file does not include a `shadow-unrelated`
error, because the new UI test file does not use `#![warn(clippy::all)]`

Merge #3389

3389: ci: when installing rust-toolchain-installer-master, install it in debug mode to save some time on ci. r=phansch a=matthiaskrgr

the compiletime optimizations probably take longer than the speedup we get when executing the optimized binary vs debug build.

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Merge pull request #3395 from phansch/add_missing_coc_file

Add missing code of conduct file

Merge pull request #3394 from phansch/rustup

Fix clippy build failure on latest master

Add missing code of conduct file

We are already using the Rust code of conduct, this just ticks off an
additional checkbox [here][community].

This version is taken from [rustfmt][rustfmt].

[community]: https://github.com/rust-lang-nursery/rust-clippy/community
[rustfmt]: https://github.com/rust-lang-nursery/rustfmt

Fix clippy build failure on latest master

Use slice patterns instead of padding

Merge #3392

3392: UI test cleanup: Extract for_loop_over_x tests r=matthiaskrgr a=phansch

cc #2038

Co-authored-by: Philipp Hansch <dev@phansch.net>

Fix dogfood and pedantic lints

Merge #3368

3368: added downsides to "known problems" for get_unwrap lint r=flip1995 a=humean

As a beginner I found this lint to be confusing because I was not sure how the `Option` type disappeared as conceptually I know that my `.get()` and Index could fail. Initially I thought maybe the compiler or clippy was smart enough to understand that it was impossible for my `.get()` to fail in this particular case, but it was explained to me that using the Index syntax is just shorthand for directly unwrapping the value:

https://doc.rust-lang.org/src/std/collections/hash/map.rs.html#1547

For beginners or users trying to iterate quickly it seems common to litter your code with `unwrap` or `except` as placeholders for where some explicit error handling might need to take place. I think it should be warned that using Index is merely more concise, but doesn't at all reduce the risk of panics and might in fact cause you to miss handling them in a future refactor.

Co-authored-by: Michael Rutter <michael.john.rutter@gmail.com>
Co-authored-by: Michael Rutter <humean@users.noreply.github.com>

ci: when installing rust-toolchain-installer-master, install it in debug mode to save some time in ci.

the compiletime optimizations probably take longer than the speedup we get when executing the optimized binary vs debug build.

Merge #3378

3378: Fix lint_without_lint_pass r=phansch a=mikerite

Co-authored-by: Michael Wright <mikerite@lavabit.com>
Co-authored-by: flip1995 <hello@philkrones.com>

RIIR update lints: Generate deprecated lints

The update script now also generates the 'register_removed' section in
`clippy_lints/src/lib.rs`.

Also, instead of using `let mut store ...`, I added a new identifier
line so that the replacement will continue to work in case `let mut
store ...` ever changes.

Merge #3370

3370: bool_comparison triggers 3 times on same code r=phansch a=mrbuzz

Fix #3335

Co-authored-by: Giorgio Gambino <gambnio.giorgio@gmail.com>

Merge #3386

3386: docs: use_self: hightlight the "should be" code sample as rust code as well. r=phansch a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Replace big if/else expression with match

Merge #3384

3384: make ci use master rustc instead of nightly r=flip1995 a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Revert "travis: work around temporary test failure due to rustc crashing on hyper."

This reverts commit 326270ad1221b54028f9d029881fa0b1fb742db9.

Revert "appveyor: use rustc nightly instead of master"

This reverts commit 3f0161918871403b4e0547191a93f395b8bf5b35.

Revert "Disable rust master toolchain build temporarily"

This reverts commit 0d899562cd805bd4335d6ee8d88e2bf1f743f000.

docs: use_self: hightlight the "should be" code sample as rust code as well.

UI test cleanup: Extract for_loop_over_x tests

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

Rustup to rustc 1.31.0-nightly (fb2446ad5 2018-10-30)

Rustup to rustc 1.31.0-nightly (fb2446ad5 2018-10-30)

 Fix #3335 rev2: bool_comparison triggers 3 times on same code

Update .stderr file

Register MISTYPED_LITERAL_SUFFIXES lint

Rewrite registered lint collection

Fix lint_without_lint_pass

Merge #3380

3380: compiletest: clean rmeta data (from "cargo check") before running compiletest r=phansch a=matthiaskrgr

Fixes #2896
Fixes #2139

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

compiletest: clean rmeta data (from "cargo check") before running compiletest.

Fixes #2896
Fixes #2139

Merge #3379

3379: dependencies: bump compiletest-rs from git to 0.3.16 r=phansch a=matthiaskrgr

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

dependencies: bump compiletest-rs from git to 0.3.16

Merge #3373

3373: UI test cleanup: Extract unnecessary_operation tests r=matthiaskrgr a=phansch

cc #2038

Co-authored-by: Philipp Hansch <dev@phansch.net>

Merge #3372

3372: UI test cleanup: Extract explicit_counter_loop tests r=matthiaskrgr a=phansch

cc #2038

Co-authored-by: Philipp Hansch <dev@phansch.net>

UI test cleanup: Extract explicit_counter_loop tests

UI test cleanup: Extract unnecessary_operation tests

Merge #3217 #3366

3217: Fix string_lit_as_bytes lint for macros r=phansch a=yaahallo

Prior to this change, string_lit_as_bytes would trigger for constructs
like `include_str!("filename").as_bytes()` and would recommend fixing it
by rewriting as `binclude_str!("filename")`.

This change updates the lint to act as an EarlyLintPass lint. It then
differentiates between string literals and macros that have bytes
yielding alternatives.

Closes #3205

3366: Don't expand macros in some suggestions r=oli-obk a=phansch

Fixes #1148
Fixes #1628
Fixes #2455
Fixes #3023
Fixes #3333
Fixes #3360

Co-authored-by: Jane Lusby <jlusby42@gmail.com>
Co-authored-by: Philipp Hansch <dev@phansch.net>

Merge #3364

3364: simplify ci scripts r=phansch a=matthiaskrgr

This makes it easier to flip the switch on pedantic lints on the clippy codebase.

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Merge pull request #3371 from matthiaskrgr/temp_breakage_fix

merge various PRs to make CI green again

appveyor: use rustc nightly instead of master

Merge branch 'compiletest' into temp_breakage_fix

pin compiletest dependency to git version (12c980f47971b5ba6beb7cb2ffebf8b32f6766ea) while we are waiting for a new release

Merge remote-tracking branch 'phansch/disable_master_build' into temp_breakage_fix

Fix #3335 rev1: bool_comparison triggers 3 times on same code

Fix #3335: bool_comparison triggers 3 times on same code

rustup: fix build with rustc 1.31.0-nightly (cae6efc37 2018-10-27)

more consistent use of terminology; trait > syntax

added downsides to "known problems" for get_unwrap lint

Disable rust master toolchain build temporarily

Don't expand macro in or_fun_call suggestion

Don't expand macro in single_match suggestion

Don't expand macro in identity_conversion suggestion

slightly simplify integration tests

simplify ci base-tests

fix: correctly reconstruct raw strings

fix: extra semicolon, only create callsite once

Fix string_lit_as_bytes lint for macros

Prior to this change, string_lit_as_bytes would trigger for constructs
like `include_str!("filename").as_bytes()` and would recommend fixing it
by rewriting as `binclude_str!("filename")`.

This change updates the lint to act as an EarlyLintPass lint. It then
differentiates between string literals and macros that have bytes
yielding alternatives.

Closes #3205

Merge #3355

3355: Lint to remove redundant `clone()`s r=oli-obk a=sinkuu

This PR adds lint `redundant_clone`. It suggests to remove redundant `clone()` that clones a owned value that will be dropped without any usage after that.

Real-world example: https://github.com/rust-lang/rust/compare/7b0735a..sinkuu:redundant_clone2

Co-authored-by: Shotaro Yamada <sinkuu@sinkuu.xyz>

Merge #3359

3359: ci: allow all branches except trying.tmp and staging.tmp to be built r=phansch a=matthiaskrgr

r? @flip1995

The problem was that with the current configuration, if I made a branch `feature1` and pushed it to travis to have it checked before making a PR, travis would skip it because it only built the 3 branches that were specified when setting up bors.

The change allows all branch names to be build by default again, except for travis.tmp.
The gh-pages branch is skipped as per travis-ci defaults: https://docs.travis-ci.com/user/customizing-the-build/#safelisting-or-blocklisting-branches

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Merge #3357

3357: Check existential types in `use_self` r=oli-obk a=HMPerson1

Fixes #3231

Co-authored-by: HMPerson1 <hmperson1@gmail.com>

Merge #3362

3362: travis: work around temporary test failure due to rustc crashing on hyper r=phansch a=matthiaskrgr

Upstream ticket: https://github.com/rust-lang/rust/issues/55376

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

travis: work around temporary test failure due to rustc crashing on hyper.

Upstream ticket: https://github.com/rust-lang/rust/issues/55376

ci: allow all branches except trying.tmp and staging.tmp to be built

Move in_macro check

Use BasicBlockData::terminator

Refactor

Use lint_root

Implement visit_basic_block_data

update_references indexing_slicing

Run update_lints

Add redundant_clone lint

Merge #3358

3358: Revert "new_ret_no_self: add sample from #3313 to Known Problems section." r=oli-obk a=matthiaskrgr

This reverts commit fd2f6dd3824b32af031d19830b6ccdc732dd3dfc.

Issue #3313 has been fixed.

Co-authored-by: Matthias Krüger <matthias.krueger@famsik.de>

Revert "new_ret_no_self: add sample from #3313 to Known Problems section."

This reverts commit fd2f6dd3824b32af031d19830b6ccdc732dd3dfc.

Issue #3313 has been fixed.

Merge #3331

3331: Disable arithmetic lints in constant items r=oli-obk a=pengowen123

Currently this will not catch cases in associated constants. I'm not sure whether checking spans is the best way to solve this issue, but I don't think it will cause any problems.
Fixes #1858

Co-authored-by: Owen Sanchez <pengowen816@gmail.com>