Auto merge of #3579 - matthiaskrgr:tool_lints, r=flip1995

update CARGO_CLIPPY_HELP string to suggest tool lints.

Auto merge of #3580 - matthiaskrgr:trivial_typos, r=flip1995

fix a couple of trivial typos (NFC).

Auto merge of #3581 - euclio:camel-case, r=flip1995

fix breakage from rust-lang/rust#57088

fix breakage from rust-lang/rust#57088

fix a couple of ftrivial typos (NFC).

update CARGO_CLIPPY_HELP string to suggest tool lints.

Auto merge of #3576 - matthiaskrgr:rustup, r=oli-obk

rustup https://github.com/rust-lang/rust/pull/56992

FIXME > TODO

rustup https://github.com/rust-lang/rust/pull/56992

Auto merge of #3573 - Vlad-Shcherbina:map_clone_caveat, r=flip1995

Document map_clone known problems #498

(cherry picked from commit ada0d2c54831a904a53ff4106e0ebb6a0f06a687)

This was lost in relicensing (057243f16b4f4233).
However, I [acknowledged](https://github.com/rust-lang/rust-clippy/issues/3099#issuecomment-416482309) relicensing so this cherry pick should be fine I guess.

Auto merge of #3341 - HMPerson1:possibly_missing_else, r=phansch

Teach `suspicious_else_formatting` about `if .. {..} {..}`

We essentially treat bare blocks `{..}` identically to `if .. {..}`, except for different lint messages.

Fixes #3044

Document map_clone known problems #498

(cherry picked from commit ada0d2c54831a904a53ff4106e0ebb6a0f06a687)

Auto merge of #3570 - muth:master, r=phansch

panic at map_unit_fn.rs:202 for map() without args

map_unit_fn.rs accessed map() arguments before type check which ensures type is Option or Result.

Boiled it down to a simple test case.

FWIW: Found this panic when running clippy against code which used a gtk::Window's map() fn inherited from gtk::WidgetExt   http://gtk-rs.org/docs/gtk/trait.WidgetExt.html#tymethod.map

Auto merge of #3569 - phansch:update_contributing, r=flip1995

Change contrib.md hierarchy, link to it from readme

'How Clippy works' and 'How to fix nightly failures' are not exactly part
of 'Writing code'.

Remove header link

test: panic at map_unit_fn.rs:202 for map() without args

rm unused file map_unit_fn.stderr

There is no map_unit_fn.rs whose output would be diffed with map_unit_fn.stderr

map_unit_fn.stderr was renamed 8 months ago from option_map_unit_fn.stderr
but option_map_unit_fn.{stderr,rs} both remain and are in use.

panic at map_unit_fn.rs:202 for map() without args

Change contrib.md hierarchy, link to it from readme

'How Clippy works' and 'How to fix nightly failures' are not exactly part
of 'Writing code'.

Workaround rust-lang/rust#43081

Teach `suspicious_else_formatting` about `if .. {..} {..}`

Auto merge of #3564 - alexcrichton:rustc-driver, r=phansch

Link to `rustc_driver` crate in plugin

This is in anticipation for rust-lang/rust#56987 where the
`rustc_driver` crate being linked in will be required to link correctly
against the compiler. In the meantime it should be harmless otherwise!

Link to `rustc_driver` crate in plugin

This is in anticipation for rust-lang/rust#56987 where the
`rustc_driver` crate being linked in will be required to link correctly
against the compiler. In the meantime it should be harmless otherwise!

Auto merge of #3566 - kinnison:kinnison/typofix, r=phansch

mutex_atomic: Correct location of AtomicBool and friends

The AtomicBool, AtomicUsize, and friends, types live in the
`std::sync::atomic` module, rather than `std::atomic` as the lint
help text used to say.

mutex_atomic: Correct location of AtomicBool and friends

The AtomicBool, AtomicUsize, and friends, types live in the
`std::sync::atomic` module, rather than `std::atomic` as the lint
help text used to say.

Auto merge of #3552 - phansch:make_integration_tests_fail_again, r=oli-obk

Make integration tests fail on 'E0463'

cc #3523

Auto merge of #3546 - matthiaskrgr:fix_install, r=oli-obk

Revert "Merge pull request #3257 from o01eg/remove-sysroot"

This reverts commit 041c49c1ed11b016d6ab9379643bb1da2adf5bfe, reversing
changes made to 1df5766cbb559aab0ad5c2296d8b768182b5186c.

The PR broke running a cargo-install'd clippy.
The installed clippy would not be able to find a crate for std.

Fixes #3523
Reopens #2874

Auto merge of #3563 - mikerite:readme-syspath, r=phansch

Update README local run command to specify syspath

Update README local run command to specify syspath

Auto merge of #3554 - klausi:module_name_repeat, r=oli-obk

chore(module_name_repeat): Rename stutter lint to module_name_repeat to avoid ableist language

See #3521

Auto merge of #3556 - lucasloisp:bool-ord-comparison, r=oli-obk

Implements lint for order comparisons against bool (#3438)

As described on issue #3438, this change implements linting for `>` and `<` comparisons against both `boolean` literals and between expressions.

Auto merge of #3557 - detrumi:allow_asref_adjustments, r=oli-obk

Fix false positive in useless_asref

Fixes #3480

Do not mark as_ref as useless if it's followed by a method call

Changes lint sugg to bitwise and operator `&`

Run update_lints after renaming

Rename lint to MODULE_NAME_REPETITIONS

Add renaming tests

Move renaming to the right place

Auto merge of #3555 - daxpedda:master, r=oli-obk

Fix `implicit_return` false positives.

Fixing some false positives on the `implicit_return` lint.

Basically it should only check for missing return statements in `block.stmts.last()` if it's a `break`, otherwise it should skip because it would either be an error, or a false positive in the case of a `loop` (which I'm trying to fix with this PR).

**Question:**
- I say "we" inside of comments ([`// make sure it's a break, otherwise we want to skip`](https://github.com/rust-lang/rust-clippy/pull/3555/files#diff-11d233fe8c8414214c2b8732b8c9877aR71)). Any alternatives or is that okay?
- I named a test [`test_loop_with_nests()`](https://github.com/rust-lang/rust-clippy/blob/6870638c3fb66c2abb20633bf40cc09ccc760047/tests/ui/implicit_return.rs#L54-L64), any better suggestions?

Implements lint for order comparisons against bool

fix(module_name_repeat): Try to register renamed lint, not valid yet

Fix an endless loop in the tests.

Fix `implicit_return` false positives.

chore(moduel_name_repeat): Rename stutter lint to module_name_repeat to avoid ableist language

Make integration tests fail on 'E0463'

base tests: make sure cargo-clippy binary can be called directly

Revert "Merge pull request #3257 from o01eg/remove-sysroot"

This reverts commit 041c49c1ed11b016d6ab9379643bb1da2adf5bfe, reversing
changes made to 1df5766cbb559aab0ad5c2296d8b768182b5186c.

Auto merge of #3545 - Kampfkarren:vec_boxed_sized, r=flip1995

Adds lint for Vec<Box<T: Sized>>

This adds, and subsequently closes #3530. This is the first time I've ever worked with anything remotely close to internal Rust code, so I'm very much unsure about the if_chain! to figure this out!

I can't get rustfmt working on WSL with nightly 2018-12-07:

`error: component 'rustfmt' for target 'x86_64-unknown-linux-gnu' is unavailable for download`

Auto merge of #3549 - flip1995:rustup, r=oli-obk

rustup rust-lang/rust#52994

`trim_left*` and `trim_right*` are deprecated as of 1.33.0.

`s/trim_left/trim_start/`
`s/trim_right/trim_end/`

rustup rust-lang/rust#52994

s/trim_left/trim_start/

s/trim_right/trim_end/

Auto merge of #3531 - mikerite:fix-3514, r=phansch

Fix write_with_newline escaping false positive

Fixes #3514

Fix test

Merge branch 'master' into fix-3514

Line length fix

Remove references to sized for end users

Remove DUMMY_SP

Add suggestion for replacement

Update lint definitions

Lint for Vec<Box<T: Sized>> - Closes #3530

Auto merge of #3535 - sinkuu:fixes, r=phansch

Fix some problems

Fixes #2892, #3199, #2841, #3476

Auto merge of #3510 - phansch:fix_doc_markdown_mixed_case, r=flip1995

Fix doc_markdown mixed case false positive

Fixes #2343

Fix doc_markdown mixed case false positive

Auto merge of #3450 - phansch:structured_sugg_for_explicit_write, r=flip1995

Add suggestion for explicit_write lint

Closes #2083

Auto merge of #3544 - flip1995:rollup, r=flip1995

Rollup of 3 pull requests

Successful merges:

 - #3509 (Fix doc_markdown off by one issue)
 - #3539 (Fix rvm/gpg bug in travis osx build)
 - #3540 (rustup https://github.com/rust-lang/rust/pull/56092)

Failed merges:

r? @ghost

Rollup merge of #3540 - matthiaskrgr:rustup, r=flip1995

rustup https://github.com/rust-lang/rust/pull/56092

fix ui test cast_alignment failure by adding #![feature(rustc_private)]

Rollup merge of #3539 - flip1995:travis_osx_fix, r=oli-obk

Fix rvm/gpg bug in travis osx build

Apparently this workaround isn't needed anymore ([travis](https://travis-ci.org/flip1995/rust-clippy/jobs/467027571)) and fixes the latest osx failures: [travis](https://travis-ci.org/rust-lang/rust-clippy/jobs/467013498#L152)

Rollup merge of #3509 - phansch:fix_doc_markdown_bugs, r=flip1995

Fix doc_markdown off by one issue

Fixes #2210

rustup https://github.com/rust-lang/rust/pull/56092

fix ui test cast_alignment failure by adding #![feature(rustc_private)]

Fix rvm/gpg bug in travis osx build

Auto merge of #3537 - matthiaskrgr:fix_script_2, r=matthiaskrgr

base-tests: don't print all commands to stdout during the loop

Fixes thinko in #3529

base-tests: don't print all commands to stdout during the loop

question_mark: Suggest Some(opt?) for if-else

redundant_field_names: Do not trigger on path with type params

Fixes #3476

question_mark: Lint only early returns

question_mark: Fix applicability

Auto merge of #3536 - matthiaskrgr:fix_script, r=oli-obk

fix typo in script

fix typo in script

Fix doc_markdown off by one issue

Remove obsolete comment

new_without_default, partialeq_ne_impl: Use span_lint_node

Fixes #2892, fixes #3199

Auto merge of #3533 - phansch:add_to_doc_valid_idents, r=phansch

Add 'CamelCase' to doc_valid_idents

Fixes #2395

Update .stderr after rebase

cargo fmt and remove stabilized feature

Make suggestion Applicability::MachineApplicable

Address review feedback

* Fix typo
* Handle None value instead of using `unwrap()`
* `pop()` instead of `x.truncate(x.len() - 1)`

Extract method

Check array lengths to prevent OOB access

Add suggestion for explicit_write lint

Auto merge of #3529 - matthiaskrgr:rustfmt_tests, r=phansch

rustfmt tests

also fails CI if tests turn out to not be formatted properly.

test reformatting: revert more questionable changes done by rustfmt and add #[rustfmt::skip]

Add 'CamelCase' to doc_valid_idents

Auto merge of #3532 - maxencefrenette:fix-contributing-md, r=oli-obk

Remove dead link in CONTRIBUTING.md

I don't think there was an issue for this, but this is just removing a dead link in CONTRIBUTING.md. The _Linting Clippy with your local changes_ section seems to have been removed without updating the table of contents.

I'm not sure if I should have opened an issue, but it seemed like a trivial fix to me.

Remove dead link in CONTRIBUTING.md

Auto merge of #3527 - phansch:update_readme2, r=matthiaskrgr

readme: tool lints are stable now

None

Fix write_with_newline escaping false positive

Fixes #3514

Merge branch 'master' into rustfmt_tests

tests: fix more cases where rustfmt would have hurt the tests

tests: revert some changs and add further rustfmt::skip attributes.

s/clippy/Clippy in readme

Also add note about using -W clippy::lint_group