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

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

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

Lint for wildcard dependencies in Cargo.toml

Merge #3339

3339: Check for known array length in `needless_range_loop` r=phansch a=HMPerson1

In `VarVisitor`, we now keep track of the type of the thing that was directly indexed and, if it's an array, check if the range's end is (or is past) the array's length.

Fixes  #3033

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

Merge #3348

3348: Setup bors for Clippy r=phansch a=flip1995

Since [bors-ng](https://app.bors.tech/repositories/3993) is already installed for this repo for a while (https://github.com/rust-lang-nursery/rust-clippy/pull/3279#issuecomment-427645938), the only thing missing, before we can use it, is the `bors.toml`. (bors-ng [docs](https://bors.tech/documentation/getting-started/))

If we want to move forward with this and this gets merged, the only thing left to do is to create the branches `staging` and `trying`.

@phansch @oli-obk

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

Merge pull request #3352 from gnieto/fix/doc-inspector

Fix inspector pass documentation

Don't emit `new_without_default_derive` if an impl of Default exists

Fix inspector pass documentation

When using `#[clippy_dump]`, the compiler complains about an unknown
attribute. The correct one seems to be `#[clippy::dump]`.

Add branch configuration to appveyor.yml

Setup bors

new_ret_no_self added test cases

new_ret_no_self walk return type to check for self

Update `ui/for_loop` test output

Check for known array length in `needless_range_loop`

new_ret_no_self correct false positive on raw pointer return types

new_ret_no_self correct linting of tuple return types

Merge pull request #3334 from matthiaskrgr/new-ret-no-self__doc

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

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

fix trivial typo on the way

Merge pull request #3303 from shssoichiro/3069-unnecessary-fold-pattern-guard

Note known false positives in unnecessary_fold lint

Merge pull request #3332 from lukasstevens/fix798

Check for comments in collapsible ifs

Support multiline comments and hopefully fix panic

Check for comments in collapsible ifs

Merge pull request #3323 from pengowen123/fix_manual_memcpy

Simplify manual_memcpy suggestion in some cases

Merge pull request #3330 from shssoichiro/3309-goblin-ice

Resolve ICE in needless range loop lint

Resolve ICE in needless range loop lint

An ICE would occur if the needless range loop was triggered
within a procedural macro, because Clippy would try to produce
a code suggestion which was invalid, and caused the compiler
to crash.

This commit takes the same approach which Clippy currently takes
to work around this type of crash in the needless pass by value lint,
which is to skip the lint if Clippy is inside of a macro.

Merge pull request #3327 from phansch/update_changelog_riir

RIIR update_lints: Update changelog links

RIIR update_lints: Update changelog links

This now also updates the link list at the bottom of the changelog.

Merge pull request #3281 from CYBAI/redundant-match

Add lint for redundant pattern matching for explicit return boolean

Merge pull request #3326 from mrbuzz/master

Reword help message for len_zero

Rename if_let_redundant_pattern_matching to redundant_pattern_matching

Also, making the old one deprecated

Add lint for redundant pattern matching for explicit return boolean

Fix issue #3322: reword help message for len_zero

Simplify manual_memcpy suggestion in some cases

Merge pull request #3325 from phansch/riir_update_lints_first_replace_region

RIIR update_lints: Replace lint count in README.md

Fix dogfood

`expect_fun_call` causes a false-positive, so I disabled it for now.

Update known problems for unnecessary_fold

Merge pull request #3321 from 0ndorio/fix/1123_false_positive_on_boxed_local

Avoid linting `boxed_local` on trait implementations.