Add `Applicability`

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

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.

RIIR update_lints: Replace lint count in README.md

This allows the usage of `util/dev update_lints` which will write the
new lint_count to the `README.md`.

Rename `active_lints` to `usable_lints`

Because now `usable_lints` will also exclude internal lints.

Merge pull request #3320 from phansch/riir_update_lints_use_walkdir

RIIR update_lints: use WalkDir instead of read_dir

Merge pull request #3319 from majecty/webpage_link_category

Website: Make lint categories linkable

Add comment on WalkDir vs. fs::read_dir

sort_by -> sort_by_key

Some more documentation for clippy_dev

Use `WalkDir` to also gather from subdirectories

`fs::read_dir` does not recurse into subdirectories.

Avoid linting `boxed_local` on trait implementations.

Website: Make lint categories linkable

Fixes #2973

out_of_bounds_indexing improved reporting of out of bounds value

Merge pull request #3316 from pengowen123/fix_needless_range_loop

Swap order of methods in `needless_range_loop` suggestion for efficiency in some cases

Restore clippy_dummy's placeholder name

Fixes #3317

Swap order of methods in `needless_range_loop` suggestion in some cases

Merge pull request #3315 from rust-lang-nursery/oli-obk-patch-1

fix for rustc master

fix for rustc master

out_of_bounds_indexing refactoring

Merge pull request #3314 from matthiaskrgr/mem_forget_sample

mem_forget: fix syntax error in code sample

mem_forget: fix syntax error in code sample

OUT_OF_BOUNDS_INDEXING fix #3102 false negative

Merge pull request #3310 from JoshMcguigan/explicit_counter_loop-3308

explicit_counter_loop fix #3308 false positive

Merge pull request #3253 from JoshMcguigan/new_ret_no_self-3220

new_ret_no_self

explicit_counter_loop fix #3308 false positive

new_ret_no_self test remove tool lints cfg flag

Added new_ret_no_self exception to clippy to pass dogfood tests

Removed new_ret_no_self tests from method.rs

new_ret_no_self correctly lint impl return

Removed unused variables

new_ret_no_self fix false positive for impl trait return with associated type self

new_ret_no_self corrected panic and added test stderr

new_ret_no_self added positive test cases

Merge pull request #3233 from rust-lang-nursery/unused-unit

new lint: unused_unit

Merge pull request #3298 from devonhollowood/pedantic-dogfood-naming

Pedantic dogfood: naming and docs

Fix some more `stutter` warnings

unused unit lint

Merge pull request #3304 from phansch/travis_windows_part2

Install Windows SDK 10.0 on travis

Install Windows SDK 10.0 on travis

Revert "Exclude pattern guards from unnecessary_fold lint"

This reverts commit d3c06f7252fdca30b19aa6ff8ecf63c86675676e.

Exclude pattern guards from unnecessary_fold lint

Methods like `Iterator::any` borrow the iterator mutably,
which is not allowed within a pattern guard and will fail to compile.
This commit prevents clippy from suggesting this type of change.

Closes #3069

Merge pull request #3291 from JoshMcguigan/cmp_owned-3289

cmp_owned wording and false positive

cmp_owned refactor

cmp_owned correct error message if rhs is deref

Merge pull request #3301 from sigustin/patch-1

Specify which categories are enabled by default

Add a comment reminding to update README if the default changes

Specify which categories are enabled by default

Closes #3293

Merge pull request #3300 from phansch/travis_windows

Add Travis windows build

Only run markdown linter on linux

Because:

* There's no need to run it on more than one platform
* It doesn't work on windows

Move Travis Windows build to allowed failures

Until the remaining issues are fixed. This also enabled `fast_finish`.
It will finish even if the windows build is still running.

Add Travis windows build

See https://blog.travis-ci.com/2018-10-11-windows-early-release

Fix `doc_markdown` lints

Fix `stutter` lints