Auto merge of #4163 - lzutao:doc-fix-foo-example, r=phansch

doc: Remove `pub` from trait definition in foo lint example

changelog: none

Auto merge of #4169 - lzutao:bump-cargo_metadata, r=Manishearth

Bump cargo metadata

Sorry, I forgot to bump `cargo_metadata` in sub-crates.

changelog: none

r? @matthiaskrgr

Bump clap in clippy_dev

Bump cargo_metadata of clippy_lints

Auto merge of #4166 - mati865:rustup, r=Manishearth

Rustup for https://github.com/rust-lang/rust/pull/61276

changelog: none

Cargo fmt

Rustup for https://github.com/rust-lang/rust/pull/61276

Auto merge of #4160 - lzutao:bump-cargo_metadata, r=matthiaskrgr

Bump cargo_metadata to 0.8.0

changelog: none

Auto merge of #4159 - lzutao:into-outer_expn_info, r=matthiaskrgr

Make use of new `outer_expn_info()` function

Find by simple command: `git grep -w expn_info`.

changelog: none

Closes #4151

doc: Remove `pub` from trait definition in foo lint example

Bump cargo_metadata to 0.8.0

Make use of new `outer_expn_info()` function

Auto merge of #4158 - phansch:rustup_tests, r=Manishearth

Fix ellipsis_inclusive_range_patterns lint warnings

Changed from `allow` to `warn` in https://github.com/rust-lang/rust/pull/61342

changelog: none

Fix ellipsis_inclusive_range_patterns lint warnings

Changed from `allow` to `warn` in https://github.com/rust-lang/rust/pull/61342

Auto merge of #4157 - phansch:update_pulldown_cmark2, r=matthiaskrgr

Update pulldown_cmark to 0.5.2

Includes various parsing fixes. Most notably [this PR][pr]

changelog: none

[pr]: https://github.com/raphlinus/pulldown-cmark/pull/325

Auto merge of #4113 - rust-lang:author, r=flip1995

Fix `Block`  dump in author lint

changelog: The `#[clippy::author]` attribute now emits correct pattern code for block expressions

Update pulldown_cmark to 0.5.2

Includes various parsing fixes. Most notably [this PR][pr]

[pr]: https://github.com/raphlinus/pulldown-cmark/pull/325

Auto merge of #4155 - phansch:rustup_trait_obj, r=oli-obk

Rustup to https://github.com/rust-lang/rust/pull/61203

See https://github.com/rust-lang/rust/pull/61203

Migrates all trait objects to use `dyn` in our ui tests

changelog: none

Rustup to https://github.com/rust-lang/rust/pull/61203

Migrates all trait objects to use `dyn`

Auto merge of #4153 - matthiaskrgr:rustup_5, r=flip1995

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

changelog: none

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

Auto merge of #4152 - matthiaskrgr:rustup_4, r=matthiaskrgr

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

(which is included in https://github.com/rust-lang/rust/pull/61274)

changelog: none

rustup https://github.com/rust-lang/rust/pull/61164
(which is included in https://github.com/rust-lang/rust/pull/61274)

Don't rustfmt author test

Co-Authored-By: Philipp Krones <hello@philkrones.com>

Fix `Block`  dump in author lint

Auto merge of #4142 - agnxy:rename-assoc, r=flip1995

Rename "Associated*" to "Assoc*"

This is to fix the breakage introduced by rust-lang/rust#60163.

changelog: none

Auto merge of #4071 - matthiaskrgr:sizeof, r=flip1995

trivially_copy_pass_by_ref: print size of type and limit in the lint message

changelog: trivially_copy_pass_by_ref: print size of type and limit in the lint message

Auto merge of #4150 - rust-lang:rustup, r=oli-obk

Rustup to rustc 1.36.0-nightly (fa40a111f 2019-05-27)

changelog: none

Rustup to rustc 1.36.0-nightly (fa40a111f 2019-05-27)

Auto merge of #4137 - euclio:let-return, r=oli-obk

tweak `let_and_return` diagnostic

changelog: `let_and_return`: label the unnecessary let binding and convert the note to a structured
suggestion

Auto merge of #4149 - flip1995:rollup-6knpdqb, r=flip1995

Rollup of 2 pull requests

Successful merges:

 - #4102 (Fix match_same_arms to fail late)
 - #4119 (Improve non ascii literal)

Failed merges:

r? @ghost

Rollup merge of #4119 - BO41:non_ascii_literal, r=flip1995

Improve non ascii literal

This PR improves the example of the [non_ascii_literal](https://rust-lang.github.io/rust-clippy/master/index.html#non_ascii_literal) lint.
It also makes it auto-fixable.

Please review. This is my first PR to this project.
(Thanks @flip1995 for the help :)

changelog: none
fixes #4117

Rollup merge of #4102 - Urriel:fix/4096_match_same_arms, r=flip1995

Fix match_same_arms to fail late

Changes:
- Add a function search_same_list which return a list of matched expressions
- Change the match_same_arms implementation behavior. It will lint each same arms found.

fixes #4096

changelog: none

Auto merge of #3832 - HarrisonMc555:use_last, r=flip1995

Implement "Use last" lint

Closes #3673

This lint checks the use of `x.get(x.len() - 1)` and suggests `x.last()` (where `x` is a `Vec`).

There's at least one more thing that needs to happen here. I believe I am correctly checking most of the scenarios and avoiding false positives. However, if different `Vec`s are used for the `x.get` and `y.len`, then it will emit a false positive. In other words, I need to check to make sure the `Vec`s are the same.

Also, a lot of these commits were temporary and not helpful to the project history...am I supposed to squash on merge? If so, how do I do that?

changelog: New lint: `get_last_with_len`

Auto merge of #4148 - sanxiyn:typo, r=Manishearth

Fix typo in comment

Fix typo in comment

Auto merge of #4145 - rust-lang:rustup, r=oli-obk

Rustup to rustc 1.36.0-nightly (2268d9923 2019-05-26)

changelog: none

Rustfmt

Rustup to rustc 1.36.0-nightly (2268d9923 2019-05-26)

Rename "Associated*" to "Assoc*"

This is to fix the breakage introduced by rust-lang/rust#60163.

Auto merge of #4141 - phansch:rustup5, r=oli-obk

Rustup to https://github.com/rust-lang/rust/pull/61026

None

Rustup to https://github.com/rust-lang/rust/pull/61026

tweak `let_and_return` diagnostic

Label the unnecessary let binding and convert the note to structured
suggestion.

Auto merge of #4135 - matthiaskrgr:rustup_obsol, r=phansch

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

changelog: none

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

Auto merge of #4134 - phansch:rustup4, r=Manishearth

Rustup to https://github.com/rust-lang/rust/pull/59545

Needed due to signature changes of `item_children` in [the PR](https://github.com/rust-lang/rust/pull/59545/files) (just grep for `item_children` to see the changes)

I'm not really sure where the .stderr changes came from.

Rustup to https://github.com/rust-lang/rust/pull/59545

trivially_copy_pass_by_ref: print size of type and limit in the lint message

Auto merge of #4132 - phansch:rollup-0vxoimc, r=phansch

Rollup of 2 pull requests

Successful merges:

 - #4129 (update if_chain to 1.0.0)
 - #4130 (bump rustc_tools_util to 0.2.0 )

Failed merges:

r? @ghost

Rollup merge of #4130 - matthiaskrgr:rtu_2, r=phansch

bump rustc_tools_util to 0.2.0

I want to push a new version to crates.io.

changelog: release rustc_tool_utils 0.2.0

Rollup merge of #4129 - matthiaskrgr:if_chain, r=phansch

update if_chain to 1.0.0

Auto merge of #4131 - phansch:rustup2, r=phansch

Rustup to https://github.com/rust-lang/rust/pull/60965

https://github.com/rust-lang/rust/pull/60965

Rustup to https://github.com/rust-lang/rust/pull/60965

clippy: bump rustc_tools util version to 0.2
rustc_tools_util: fix typo in docs (readme)

rustc_tool_utils: bump version to 0.2.0

update if_chain to 1.0.0

Auto merge of #4128 - matthiaskrgr:test_update, r=phansch

tests: update needless_bool test stderr

I'm getting a test failure otherwise with rust master (f688ba608923bdbf6b46ec65af2f6464b6233a75).

changelog: none

tests: update needless_bool test stderr

I'm getting a test failure otherwise with rust master (f688ba608923bdbf6b46ec65af2f6464b6233a75).

Auto merge of #4126 - phansch:rustup, r=Manishearth

Rustup to https://github.com/rust-lang/rust/pull/60740

https://github.com/rust-lang/rust/pull/60740

changelog: none

cargo fmt

Roses are red
Violets are blue
`Add rustfmt to a pre-commit hook` is what I should do.

Rustup to https://github.com/rust-lang/rust/pull/60740

changelog: none

Auto merge of #4120 - phansch:needless_lifetimes_async_test_cleanup, r=Manishearth

Lifetimes UI test cleanup

* Removes an unneeded `#![warn(clippy::extra_unused_lifetimes)]` from
  the `needless_lifetimes` test - the tests for those are in a different
  file.

* Renames `unused_lt.{rs,stderr}` to `extra_unused_lifetimes.{rs,stderr}`
* Renames `lifetimes.{rs,stderr}` to `needless_lifetimes.{rs,stderr}`

Auto merge of #4125 - Manishearth:rustup, r=oli-obk

Rustup to rustc 1.36.0-nightly (dbfe70dfc 2019-05-22)

not 100% sure of the Symbol stuff

r? @oli-obk

Disable cargo integration test due to clippy ICE

Rustup to rustc 1.36.0-nightly (dbfe70dfc 2019-05-22)

Fix breaking tests

Changes:
- Fix stderr breaking the tests
- Adding tests over the if arms

Add the common case search

Changes:
- Refactor the common case search into a function.
- Fix the useless Option around the vec in the search_same_list.

Formatting inside if_chain! macro

Implement get_last_with_len lint

Lifetimes UI test cleanup

* Removes an unneeded `#![warn(clippy::extra_unused_lifetimes)]` from
  the `needless_lifetimes` test - the tests for those are in a different
  file.

* Renames `unused_lt.{rs,stderr}` to `extra_unused_lifetimes.{rs,stderr}`
* Renames `lifetimes.{rs,stderr}` to `needless_lifetimes.{rs,stderr}`

Fix tests and make other ascii lints auto-fixable

Auto merge of #4118 - rust-lang:revert, r=phansch

Revert "Turn off two tests broken due to #4108"

This reverts commit 568a3ecfc3eebe099c1ffe44c3d9afdc5d445fd2 which is unnecessary after https://github.com/rust-lang/rust-clippy/pull/4115

changelog: none

Make non_ascii_literal auto-fixable

Improve verbosity of non_ascii_literal lint example

Revert "Turn off two tests broken due to #4108"

This reverts commit 568a3ecfc3eebe099c1ffe44c3d9afdc5d445fd2.

Auto merge of #4088 - pJunger:check1, r=oli-obk

Added lint for TryFrom for checked integer conversion.

works towards #3947

Added lint for try_from for checked integer conversion.
Should recognize simple & straight-forward checked integer conversions.

Auto merge of #4115 - ehuss:fix-compiletest, r=Manishearth

Fix compile-test from forcing a rebuild.

If the `compile-test` test was run, then running a cargo build command immediately after caused everything to be rebuilt. This is because the `compile-test` test was deleting all `.rmeta` files in the target directory. Cargo recently switched to always generating `.rmeta` files (https://github.com/rust-lang/cargo/pull/6883), and when the files are deleted, it thinks it needs to be rebuilt.

I am not very familiar with compiletest or clippy, so please take a close look and test this out (with the most recent nightly). In particular, make sure it doesn't revert the fixes from #3380 (it seems to work for me). Also @oli-obk mentioned something related in https://github.com/rust-lang/rust/pull/60190#issuecomment-493617188, and I want to make sure that is addressed as well.

Fixes #4114

Fix compile-test from forcing a rebuild.

Auto merge of #4109 - Manishearth:backport-merge, r=phansch

Merge backport branches into master

Attempt 2 at https://github.com/rust-lang/rust-clippy/pull/4103

This does nothing to the code, however it keeps the backport branches in master history, so that deleting a branch won't accidentally break old rustc builds

Auto merge of #4107 - taiki-e:useless_attribute, r=flip1995

useless_attribute: Add unreachable_pub to whitelists

Fixes #4106

changelog: `useless_attribute`: whitelist `unreachable_pub` on `use` items

Auto merge of #4099 - flip1995:ul_4094, r=oli-obk

Add macro check for unreadable_literal lint

Closes #4094

changelog: Disable `unreadable_literal` lint inside macros

Auto merge of #4100 - phansch:add_stderr_length_check, r=flip1995

Add a stderr file length check to clippy_dev

This adds a check to `clippy_dev` that enforces a maximum line count for
`stderr` files. CI will fail if the line count is exceeded. It's
currently set to `320` lines.

Ideally this would be implemented in `compiletest-rs` but there are
plans to move Rust's `compiletest` into the `compiletest-rs` repository
and I don't want to do the work in `compiletest` twice. However, I also
don't want to wait until the move is done, so I added the check to
`clippy_dev` until it makes sense to add it to compiletest-rs.

cc #2038

Auto merge of #4111 - Manishearth:changelog, r=oli-obk

Update changelog

r? @oli-obk @phansch

Update master changelog

Up to 60a609acaed3bf2b3ec6ab995bccf0f03bc26060

Update beta changelog

Updated README.

Removed symbols again.

Removed unused lifetime.

Changed impl to use symbols.

Added suggestion for conversion with is_ok.

Fixed more lint findings.

Fixed clippy lints in checked_conversions.rs.

Registered lint.

Added lint for TryFrom for checked integer conversion rust-lang#3947.

useless_attribute: Add unreachable_pub to whitelists

Auto merge of #4110 - rust-lang:symbolic_wasteland, r=oli-obk

Prevent symbocalypse

r? @Manishearth

This is strictly better, we can just not modify rustc and bump solely the clippy submodule and then implement diagnostic lang items without beta looming over us

changelog: none

Dogfood

Dogfood and rustfmt

Turn off two tests broken due to #4108

Prevent symbocalypse