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

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

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

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

Fix `similar_names` warnings

Most of these are just `#![allow]`ed, because they are things like using
l vs r to differentiate left vs right. These would be made less clear by
taking the advice of `similar_names`

cmp_owned current suggestion for multiple deref

Remove now-useless `allow(unknown_lints)`

Stabilize tool lints

Merge pull request #3294 from mikerite/fix-3276

Fix fn_to_numeric_cast_with_truncation suppression

Merge pull request #3295 from phansch/refactor_clippy_dev

Use `impl Iterator` in arg position in clippy_dev

Use `impl Iterator` in arg position in clippy_dev

Small refactoring pulled out of work on #3266.
This should make the methods a bit more flexible.

Merge pull request #3288 from devonhollowood/pedantic-dogfood-casts

Pedantic dogfood: casts

Fix fn_to_numeric_cast_with_truncation suppression

Fixes #3276

Merge pull request #3127 from mikerite/fix-2937

Fix 2937

Merge pull request #3292 from kimsnj/commutative_assign_op

Limit commutative assign op lint to primitive types

Limit commutative assign op lint to primitive types

cmp_owned add test for multiple dereference

Clarify code

Take advantage of the fact that very large regexes are unlikely

Fix #2937

Corrected single-character string constant used as pattern found in dogfood test

Fixes 3289, cmp_owned wording and false positive

Merge pull request #3287 from JoshMcguigan/cmp_owned-2925

cmp_owned false positive

Fix cast_possible_wrap and cast_sign_loss warnings

Fix cast_possible_truncation warnings

Fixes #2925 cmp_owned false positive

Merge pull request #3278 from d-dorazio/fix-contributing-manual-test-command

contributing.md: fix command to manually test an example

Merge pull request #3234 from matthiaskrgr/fix_3064_crash

if_let_redundant_pattern_matching: use Span.to() instead of Span.with_hi() to fix crash.

if_let_redundant_pattern_matching: use Span.to() instead of Span.with_hi() to fix crash.

Fixes #3064

Merge pull request #3286 from vmalloc/lifetime_elision_msg

Improve diagnostics in case of lifetime elision (closes #3284)

Improve diagnostics in case of lifetime elision (closes #3284)

Merge pull request #3282 from JoshMcguigan/excessive_precision-2840

Fix excessive_precision false positive

Merge pull request #3265 from mikerite/fix-export

Fix util/export.py to include lints from methods

Merge pull request #3283 from etaoins/dont-suggest-cloned-for-map-box-deref

Don't suggest cloned() for map Box deref

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