Add failing test

Reinserted commata

Recomend `.as_ref()?` in certain situations

Deduplicate some code?

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 `&`

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 an endless loop in the tests.

Fix `implicit_return` false positives.

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

readme: tool lints are stable now

Auto merge of #3511 - phansch:remove_allow_doc_markdown, r=phansch

Remove allow(doc_markdown) in excessive_precision.rs

Closes #3299

Auto merge of #3518 - sinkuu:redundant_clone_tw, r=phansch

Lint redundant clone of fields

Makes `redundant_clone` warn on unnecessary `foo.field.clone()` sometimes (it can detect an unnecessary clone only if the base of projection, `foo` in this case, is not used at all after that). This is enough for cases like `returns_tuple().0.clone()`.

test formatting: don't format tests/ui/formatting.rs

update test stderr

Auto merge of #3524 - phansch:update_contributing, r=phansch

Small updates to CONTRIBUTING.md

None

Document bors/homu

Small updates to CONTRIBUTING.md

Only check the assignment found at last

If there are more than one such assignment, the last one may be
the one supplied to `clone` method.
Makes `find_stmt_assigns_to` internally reverses the iterator to make
the intent to "iterate statements backward" clear.

Add comment and rename

Auto merge of #3522 - matthiaskrgr:target_dir, r=phansch

travis: base-tests: share CARGO_TARGET_DIR between check runs of subcrates to avoid unnecessarily recompiling deps.

Apply redundant_clone on clippy

Lint redundant clone of projection

rustfmt tests

travis: base-tests: share CARGO_TARGET_DIR between check runs of subcrates to avoid unneccessarily recompiling deps.

base tests: assert that tests are properly formatted.

Auto merge of #3520 - matthiaskrgr:test_rustfmt_skip, r=phansch

add rustfmt::skip attributes to a couple of tests

part of rustfmt'ing test code

update line numbers of tests

add rustfmt::skip attributes to some tests

Merge pull request #3517 from phansch/add_fast_finish

Add `fast_finish` to travis matrix

Merge pull request #3516 from phansch/remove_run_rustfix_annotation

Remove a run-rustfix annotation (for now)

Fix test

`if true` is recognized by MIR optimization.

Add `fast_finish` to travis matrix

This means we don't have to wait for `allowed_failures` builds to
complete. It should save us ~10 minutes until we remove the windows
build from `allowed_failures`.

Remove a run-rustfix annotation (for now)

Starting to work on #2376, this annotation got in the way. Going to
remove it for now.

Merge pull request #3515 from matthiaskrgr/fmt

run rustfmt

run rustfmt

Cleanup

Merge pull request #3512 from matthiaskrgr/rustup

rustup https://github.com/rust-lang/rust/pull/56502 ( .hir -> .hir() )

s/rustfmt-preview/rustfmt/

Swap if branches

Fix c_void false positive caused by libc refactoring

The path of `libc::c_void` has changes in https://github.com/rust-lang/libc/commit/5c1a6b8a6d558882927a0816d91c01b9c2a88018
The DefId path is now always platform specific like
`libc::windows::c_void`. This fixes our c_void detection to only check
the first and last elements.

rustup https://github.com/rust-lang/rust/pull/56502 ( .hir -> .hir() )

Merge pull request #3336 from HMPerson1/clone_on_copy_deref

Fix `clone_on_copy` not detecting derefs sometimes

Remove allow(doc_markdown) in excessive_precision.rs

Merge pull request #3507 from flying-sheep/patch-1

typo: emum → enum