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>

Merge #3346

3346: Add lint for calling `mem::discriminant` on a non-enum type r=flip1995 a=HMPerson1

Also, if the type is a reference to an enum, we suggest removing `&`s and/or dereferencing.

Fixes #3342

Co-authored-by: HMPerson1 <hmperson1@gmail.com>
Co-authored-by: Philipp Krones <hello@philkrones.com>

Add tests for more than one level of reference

Add test case for `mem::discriminant` inside a macro

Add `Applicability`

Add lint for calling `mem::discriminant` on a non-enum type

Check existential types in `use_self`

Disable arithmetic lints in constant items

Merge #3312

3312: OUT_OF_BOUNDS_INDEXING false negative r=phansch a=JoshMcguigan

fixes #3102

Co-authored-by: Josh Mcguigan <joshmcg88@gmail.com>

Merge #3356

3356: Fix warnings introduced by #3349 r=flip1995 a=flip1995

I missed these warnings during review, should have checked the Travis log first.

Co-authored-by: flip1995 <hello@philkrones.com>

Merge #3338

3338: new_ret_no_self false positives r=flip1995 a=JoshMcguigan

~~WORK IN PROGRESS~~

I plan to fix all of the false positives in #3313 in this PR, but I wanted to open it now to start gathering feedback.

In this first commit, I've updated the lint to allow tuple return types as long as `Self` shows up at least once, in any position of the tuple. I believe this is the broadest possible interpretation of what should be allowed for tuple return types, but I would certainly be okay making the lint more strict.

fixes #3313

Co-authored-by: Josh Mcguigan <joshmcg88@gmail.com>

Fix warnings introduced by #3349

new_ret_no_self added test cases

Merge #3349

3349: Fixes #3347: Lint for wildcard dependencies in Cargo.toml r=ordovicia a=ordovicia

Add a lint for wildcard dependencies in Cargo.toml.
How should I write a test for this lint?

Fixes #3347

Co-authored-by: Hidehito Yabuuchi <hdht.ybuc@gmail.com>

Replace remaining `krate.span` with `DUMMY_SP`

Use DUMMY_SP in multiple_crate_versions

Some fixes for wildcard_dependencies

Merge #3350

3350: Don't emit `new_without_default_derive` if an impl of Default exists regardless of generics r=oli-obk a=pengowen123

Fixes #2226

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

Run util/update_lints.py

Minor changes on clippy_lints/src/wildcard_dependencies.rs