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

Rustup to rustc 1.36.0-nightly (1764b2972 2019-05-12)

Rustfmt all the things

Clippy dogfood

Update for compiletest changes

Use symbols instead of strings

Rustup to rustc 1.36.0-nightly (1764b2972 2019-05-12)

Auto merge of #4089 - flip1995:id_conv_reg_test, r=Manishearth

Add regression test for identity_conversion FP

cc #3913 #4082 #3936

changelog: none

Add regression test for identity_conversion FP

Auto merge of #4087 - phansch:move_tests, r=matthiaskrgr

UI test cleanup: Extract many_single_char_names tests

changelog: none

cc #2038, #4086

UI test cleanup: Extract many_single_char_names tests

Auto merge of #4085 - phansch:empty_loop_tests, r=matthiaskrgr

Add tests for empty_loop lint

changelog: none

Closes #4072

Add tests for empty_loop lint

Auto merge of #4082 - Manishearth:macro-check-split, r=oli-obk

Make most macro checks also check for desugarings

We should audit the macro checks one by one and re-add `in_macro`. I suspect it's applicable to most of them.

fixes https://github.com/rust-lang/rust-clippy/issues/4081

Add in_macro again

Rename in_macro to in_macro_or_desugar

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

Rustup to rustc 1.36.0-nightly (acc7e652f 2019-05-10)

Fixes breakages from https://github.com/rust-lang/rust/pull/59288

Not finished yet, help appreciated.

Todo:

 - [x] Needs to build
 - [x] Util should handle DropTemps, https://github.com/rust-lang/rust-clippy/pull/4080#discussion_r283079202
 - [x] Author lint should spit out higher::if_block checks
 - [x] Unsure what to do in consts.rs
 - [x] Needs to pass tests

Start handling desugarings in author lint

Ignore desugarings in macro checks

Add IfDesugar to desugaring_name

Fix consts.rs

More uses of higher::if_block

Remove some unnecessary If arms

Fix needless_bool.rs

Fix question_mark.rs

Fix shadow.rs

Fix unwrap.rs

Add util function for desugaring if block

Auto merge of #4073 - euclio:serde-derive, r=Manishearth

use `derive` feature over `serde_derive` crate

changelog: replace `serde_derive` dependency with `serde`'s derive feature

use `derive` feature over `serde_derive` crate

Auto merge of #4069 - mikerite:while_loop_test_split, r=phansch

Reorganize "while loop" tests

cc #2038

changelog: none

Remove unused attrs in while_let_on_iterator tests

Fix while_let_on_iterator.stderr

Merge branch 'master' into while_loop_test_split

Rename while_loop tests to while_let_on_iterator

Split while_loop.rs tests

Split out while_let_loop tests into separate file.
Split ice-360 out into separate file.

Auto merge of #4068 - matthiaskrgr:rustup_3, r=Manishearth

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

cc https://github.com/rust-lang/rust/issues/60623

changelog: none

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

cc https://github.com/rust-lang/rust/issues/60623

Auto merge of #4064 - mikerite:fix-771, r=matthiaskrgr

Add test for #771.

Closes #771

<!--
Thank you for making Clippy better!

We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.

If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.

If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.

- [ ] Followed [lint naming conventions][lint_naming]
- [ ] Added passing UI tests (including committed `.stderr` file)
- [ ] `cargo test` passes locally
- [ ] Executed `util/dev update_lints`
- [ ] Added lint documentation
- [ ] Run `cargo fmt`

Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.

Delete this line and everything above before opening your PR -->

changelog: none

Add test for #771.

Closes #771

Auto merge of #4063 - mikerite:while_loop_clean_up, r=Manishearth

Clean up while_loop tests

At some stage, we started adding test cases inside the function which tests
refutable patterns. This commits splits those test cases out into separate
functions.

Clean up while_loop tests

At some stage, we started adding test cases inside the function which tests
refutable patterns. This commits splits those test cases out into separate
functions.

Auto merge of #4061 - rust-lang:rustup, r=phansch

Rustup to rustc 1.36.0-nightly (13fde05b1 2019-05-03)

Trying to deal with changes from https://github.com/rust-lang/rust/pull/60462

Moved from https://github.com/rust-lang/rust-clippy/pull/4060 so everyone can collab on the rustup branch.

Rustup to rustc 1.36.0-nightly (13fde05b1 2019-05-03)

Auto merge of #4059 - rustic-games:into_iter_on_array/doc-fix, r=flip1995

Fix link in into_iter_on_array documentation

The non-inline variant wasn't being rendered correctly.

see: https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_array

Fix link in into_iter_on_array documentation

The non-inline variant wasn't being rendered correctly.

see: https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_array

Auto merge of #4053 - euclio:too-many-args-span, r=flip1995

remove function body from "too many args" span

changelog: Remove the function body from the "too many arguments" span.

remove function body from "too many args" span

Auto merge of #4049 - airt:fix-4033-search_is_some, r=flip1995

Fix #4033 search_is_some

Fixes #4033.

Suggest `any(|x| ..)` instead of `any(|&x| ..)` for `find(|&x| ..).is_some()` (Lint [search_is_some](https://rust-lang.github.io/rust-clippy/master/index.html#search_is_some))

FnDecl of `find`:

```rust
fn find<P>(&mut self, mut p: P) -> Option<Self::Item> where
    P: FnMut(&Self::Item) -> bool
```

FnDecl of `any`:

```rust
fn any<F>(&mut self, mut f: F) -> bool where
    F: FnMut(Self::Item) -> bool
```

If match on `|&_|` in closure of `find`, only use `|_|` in the suggestion.

PS. It's the first time that I have used the `hir` API, please correct me if there is any mistake 😺

Auto merge of #4035 - JoshMcguigan:useless_let_if_seq-3043, r=oli-obk

useless_let_if_seq handle interior mutability

fixes #3043

This passes all tests, including a new one specifically dealing with a type with interior mutability. The main thing I'm unsure of is whether the span I used in the call to `is_freeze` is the most appropriate span to use, or if it matters.

Auto merge of #4056 - matthiaskrgr:rustup__, r=Manishearth

 more Use->DropTemps fixes

changelog: none

more Use->DropTemps fixes

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

cc https://github.com/rust-lang/rust/issues/60448

Auto merge of #4048 - matthiaskrgr:driver_version, r=phansch

clippy-driver: use rustc_tools_util to get version info.

changelog: make "clippy-driver -V" output the same information as "cargo clippy -V"

clippy-driver: use rustc_tools_util to get version info.

This will add git hash information to `clippy-driver -V` output.

Auto merge of #4051 - yaahallo:master, r=Manishearth

use clippy-preview instead of clippy in dogfood

changelog: switch dogfood tests to use cargo clippy-preview

closes #4050

use clippy-preview instead of clippy in cargo

check closure arguments before use it

format code

Auto merge of #4039 - andrehjr:add-find-map-lints, r=flip1995

Add lints for find_map

changelog: adds lints for find_map and filter_map_next. Closes issue #3474

Hope I got everything correctly this time! Let me know if I missed something.

Update find(p).map(p) occurrences to use find_map(p)

Add lints for find_map

fix suggestion for search_is_some

fix suggestion for search_is_some naively

change |&x| to |x| in stderr file

Auto merge of #4045 - matthiaskrgr:RTU, r=phansch

rustc_tools_util: try to handle case of not having CFG_RELEASE_CHANNEL better when getting compiler channel.

changelog: rustc_tools_util: try to handle case of not having CFG_RELEASE_CHANNEL better when getting compiler channel.

Auto merge of #4047 - rust-lang:rustup, r=Centril

Rustup to rustc 1.36.0-nightly (f843ad60e 2019-04-30)

None

Rustup to rustc 1.36.0-nightly (f843ad60e 2019-04-30)

Auto merge of #4032 - phansch:add_tests, r=phansch

Add two more tests for redundant_closure

These two cases were fixed by #4008.

Closes #1439

changelog: none

Auto merge of #4046 - rust-lang:rustup, r=phansch

Rustup to rustc 1.36.0-nightly (c7fcbfbf1 2019-04-29)

Updates to pass the lint introduced in https://github.com/rust-lang/rust/pull/60317

r? @oli-obk @mikerite

Rustup to rustc 1.36.0-nightly (c7fcbfbf1 2019-04-29)

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

Auto merge of #4044 - matthiaskrgr:travis_sort_jobs, r=phansch

travis: sort integration test jobs so that the jobs taking the most time are run first

changelog: none

rustc_tools_util: try to handle case of not having CFG_RELEASE_CHANNEL better when getting compiler channel.

travis: sort integration test jobs so that the jobs taking the most time are run first

Auto merge of #4043 - rust-lang:map-clone-iter, r=matthiaskrgr

Suggest .copied() for map_clone on iterators too

fixes https://github.com/rust-lang/rust-clippy/issues/3958

changelog: Make `map_clone` suggest the newly-stable `Iterator::copied()` when applicable

r? @mikerite @matthiaskrgr

Suggest .copied() for map_clone on iterators too

Auto merge of #4042 - matthiaskrgr:toml_update, r=Manishearth

deps: bump toml from 0.4 to 0.5

changelog: none

deps: bump toml from 0.4 to 0.5

Auto merge of #4040 - mikerite:fix-build-20190428, r=Manishearth

Fix breakage due to rust-lang/rust#60225

Wrote this up before I saw that Manish already started on a fix in #4038. It no doubt contains errors. Feel free to close.

Fix breakage due to rust-lang/rust#60225

Add two more tests for redundant_closure

These two cases were fixed by #4008.

Closes #1439

changelog: none

cargo fmt

useless_let_if_seq handle interior mutability

Auto merge of #4008 - g-bartoszek:boxed-fnmut, r=phansch

Do not trigger redundant_closure for non-function types

fixes #3898

Added a check for the entity being called in the closure body to be a FnDef. This way lint does not trigger for ADTs (Box) but I'm not sure if it's correct and not too restrictive.

<!--
Thank you for making Clippy better!

We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.

If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.

If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.

- [ ] Followed [lint naming conventions][lint_naming]
- [ ] Added passing UI tests (including committed `.stderr` file)
- [ ] `cargo test` passes locally
- [ ] Executed `util/dev update_lints`
- [ ] Added lint documentation
- [ ] Run `cargo fmt`

Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.

Delete this line and everything above before opening your PR -->

changelog: Fix false positive in `redundant_closure` pertaining to non-function types

Auto merge of #4026 - cemiloten:working-on-#3981, r=flip1995

Attempt to fix #3981

Fixes #3981

Hi, hopefully this is correct, happy to have feedback.

changelog: Don't trigger `unnecessary_cast` inside a macro

Ignore unnecessary cast if inside macro

Auto merge of #4029 - phansch:update_pulldown_cmark, r=oli-obk

Update pulldown_cmark to 0.5

We now no longer have to use our own wrapper around `Parser` and can use
the new `OffsetIter`.

changelog: none

Auto merge of #4028 - phansch:skip_appveyor_on_pr_pushes, r=phansch

Only run AppVeyor on r+, try and the master branch

As it is right now, there is only one worker available in the `rust-lang-libs`
AppVeyor project and there are other repos as well that we share this worker
with. This has been a problem for us because we sometimes hit a bors timeout if there
are too many builds queued up.

To improve the situation, I think we could try to use AppVeyor a bit less
often. The average PR is not going to break windows related things anyway, so
it should be fine to run it on r+/try/master only.

changelog: none

Auto merge of #4030 - phansch:rustup190425, r=Manishearth

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

changelog: none

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

Update pulldown_cmark to 0.5

We now no longer have to use our own wrapper around `Parser` and can use
the new `OffsetIter`.

Auto merge of #4027 - kraai:remove-approx_constant-known-problems, r=phansch

Remove approx_constant known problems

Fixes #4025.

changelog: Remove `approx_constant`'s documentation's "Known problems" section.

Only run AppVeyor on r+, try and the master branch

As it is right now, there is only one worker available in the `rust-lang-libs`
AppVeyor project and there are other repos as well that we share this worker
with. This has been a problem for us because we sometimes hit a bors timeout if there
are too many builds queued up.

To improve the situation, I think we could try to use AppVeyor a bit less
often. The average PR is not going to break windows related things anyway, so
it should be fine to run it on r+/try/master only.

Auto merge of #4023 - phansch:add_test, r=flip1995

Add test for or_fun_call macro suggestion

changelog: none

Closes #1018

Remove approx_constant known problems

Fixes #4025.

Auto merge of #4022 - phansch:move_some_loop_tests, r=flip1995

UI test cleanup: Extract needless_range_loop tests

changelog: none

Auto merge of #4024 - kraai:suppress-let_and_return-if-let-has-attributes, r=flip1995

Suppress let_and_return if let has attributes

Fixes #3882.

changelog: suppress `let_and_return` if `let` has attributes

Suppress let_and_return if let has attributes

Fixes #3882.

Add test for or_fun_call macro suggestion

Closes #1018

UI test cleanup: Extract needless_range_loop tests

Auto merge of #4021 - kraai:change-cast_lossless-warning, r=Manishearth

Change "if types change" to "if you later change the type"

Fixes #3964.

changelog: change wording of `cast_lossless` message