Merge remote-tracking branch 'upstream/rust-1.36.0' into backport_merge

Auto merge of #4747 - matthiaskrgr:rustup_19, r=flip1995

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

changelog: none

travis: temporarily disable rustfmt ci check until #4742 is resolved

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

Auto merge of #4739 - flip1995:rustup, r=flip1995

Rustup to rust-lang/rust#65773

changelog: none

Rustup to rust-lang/rust#65773

Auto merge of #4736 - flip1995:icu_mul_add, r=matthiaskrgr

Move manual_mul_add into nursery

Addresses #4735

changelog: Move [`manual_mul_add`] into nursery

Move manual_mul_add into nursery

Auto merge of #4733 - lzutao:how-rtim, r=flip1995

build: install RTIM if it doesn't exist and upgrade it if it is in cargo bin home

Replace #4731
changelog: none

build: install RTIM if it doesn't exist and upgrade it if it is in cargo bin home

Auto merge of #4724 - lzutao:minor-appveyor, r=phansch

appveyor: minor syntax change

Use this link to review diff: https://github.com/rust-lang/rust-clippy/pull/4724/files?w=1
changelog: none

appveyor: Leading space change only

appveyor: Minor config change

Auto merge of #4721 - phansch:fix_try_err_in_ext_macro, r=flip1995

Don't emit try_err lint in external macros

changelog: Fix [`try_err`] false positive in external macros

Closes #4709

Auto merge of #4713 - lzutao:no-reinstall-toolchain, r=phansch

build: do not reinstall master toolchain if it is up-to-date

changelog: none

build: do not reinstall master toolchain if it is up-to-date

Auto merge of #4650 - Mark-Simulacrum:clippy-up-lintstore-lockless, r=phansch

Update clippy for latest rustc changes

Specifically, this revises the clippy integration to utilize a new
callback to register its lints, as the prior editing of lint store in
Session is no longer possible.

---

changelog: none

Fix dogfood findings

Revert "rustc-dev has been temporarily removed"

This reverts commit 974a05b806954ebdc6640b7ef539667bd7baca56.

Fix lint_without_lint_pass lint

Fix tests

Run ./util/dev fmt

Rustup to rust-lang/rust#65657

Update clippy_dev

Update clippy for latest rustc changes

Specifically, this revises the clippy integration to utilize a new
callback to register its lints, as the prior editing of lint store in
Session is no longer possible.

Don't emit try_err lint in external macros

Auto merge of #4675 - lzutao:improve-shellscript, r=phansch

build: improve script and travis config

* fix a diff failure on windows

See https://travis-ci.com/rust-lang/rust-clippy/jobs/245971932#L1625
for an example.

* use cmp instead of diff > /dev/null

* clone single branch instead of clone then checking out

* do not decrypt key if have no diff change

changelog: none

Auto merge of #4680 - hellow554:debug_assert_mut_call, r=flip1995

Add lint for debug_assert_with_mut_call

closes #1526

**What does not work:**

* detecting a mut call in the format string itself, e.g. `debug_assert!(false, "{}", vec![1].pop())`
* detecting `*mut T` usage (pointer)

---

changelog: add new lint `debug_assert_with_mut_call`

Auto merge of #4691 - HMPerson1:suggest_iter, r=phansch

Fix suggestion of `explicit_counter_loop`

changelog: In the suggestion of `explicit_counter_loop`, if the `for` loop argument doesn't implement `Iterator`, then we suggest `x.into_iter().enumerate()` (or `x.iter{_mut}()` as appropriate). Also, the span of the suggestion has been corrected.

Fixes #4678

Auto merge of #4718 - matthiaskrgr:cargo_metadata, r=phansch

deps: update cargo_metadata from 0.8.0 to 0.9.0

changelog: none

deps: update cargo_metadata from 0.8.0 to 0.9.0

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

Rustup to rust-lang/rust#65647

cc rust-lang/rust#65647

waiting on rust-lang/rust#65690

changelog: none

Use StableHasher in SpanlessHasher

Don't hash spans in SpanlessHasher

build: remove needless BASE_TESTS env

Early exit if errors in integration tests

build: remove travis_wait

Now the travis CI build time is fast enough to not cause build timeout.

nit: ! is not on a condition and skips errexit

build: use shared target dir

Add lint for debug_assert_with_mut_call

This lint will complain when you put a mutable function/method call
inside a `debug_assert` macro, because it will not be executed in
release mode, therefore it will change the execution flow, which is not
wanted.

macos: symlink rustc dev libs to /usr/local/lib

Due to System Integrity Protection (SIP), exporting
LD_LIBRARY_PATH will not work on macOS systems.
Therefore, rather than disable SIP, we could symlink
those rust dev libs to /usr/local/bin.
I already tried install_name_tool(1) but couldn't make
clippy-driver work on CI for whatever reasons.

build: improve script and travis config

* fix a diff failure on windows

See https://travis-ci.com/rust-lang/rust-clippy/jobs/245971932#L1625
for an example.

* use cmp instead of diff > /dev/null

* clone single branch instead of clone then checking out

* do not decrypt key if have no diff change

Auto merge of #4702 - lzutao:build/debug-integration-clippy, r=phansch

build: use clippy's debug build in integration test

changelog: none

build: use clippy's debug build in integration test

Auto merge of #4692 - lzutao:use-opt-profile-rtim, r=phansch

build: use release build of RTIM

rustup-toolchain-install-master on Travis Windows builds is
unexpectedly slower compared to when run on other OSes.
This commit tries to use release build of RTIM as a mean to
improve performance there.

changelog: none

Auto merge of #4700 - lzutao:rustup-65535, r=phansch

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

cc https://github.com/rust-lang/rust/issues/65585
changelog: none

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

Auto merge of #4696 - rust-lang:need-not-use-proc-macro, r=phansch

Omit proc macros from `must_use_candidate`

This fixes #4684.

changelog: none

Fix suggestion for ranges

Omit proc macros from `must_use_candidate`

Auto merge of #4657 - Licenser:additional-restrictions, r=flip1995

Additional restrictions

Add restriction lints for `panic!`, `unreachable!`, `todo!` and `.expect(...)`

changelog: Add 5 new `restriction` lints: `panic`, `unreachable`, `todo`, `option_expect_used`, `result_expect_used`

build: use release build of RTIM

rustup-toolchain-install-master on Travis Windows builds is
unexpectedly slower when it was running on other OSes.
This commit tries to use release build of RTIM as a mean to
improve performance there.

Split out tests

Add expect

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

Add todo and tests

add restirction for unreachable and panic

Fix suggestion span in `explicit_counter_loop`

Auto merge of #4687 - lzutao:before-cache-appveyor, r=Manishearth

appveyor: on_finish run after saving cache

According to <https://www.appveyor.com/docs/build-configuration/#build-pipeline>
on_finish step runs after saving cache step, we should use after_test instead.

changelog: none

Suggest calling `iter` if needed in `explicit_counter_loop`

Auto merge of #4685 - HMPerson1:patch-1, r=Manishearth

Update adding_lints.md

changelog: none

Auto merge of #4686 - lzutao:fix-travis-windows-stuck, r=Manishearth

travis: Windows build might be stuck as non-supported languages

See <https://docs.travis-ci.com/user/reference/windows/#supported-languages>
for list of Windows supported languages.

I'm sorry for making Travis Windows build stuck.

changelog: none

Auto merge of #4683 - HMPerson1:inefficient_to_string, r=Manishearth

Add `inefficient_to_string` lint

Closes #4586

changelog: Add `inefficient_to_string` lint, which checks for calling `to_string` on `&&str`, which would bypass the `str`'s specialization

Update help text in `inefficient_to_string`

Co-Authored-By: Manish Goregaokar <manishsmail@gmail.com>

Auto merge of #4689 - Manishearth:rustup, r=Yaahc

Rustup to rustc 1.40.0-nightly (fa0f7d008 2019-10-17)

changelog: none

syntax::ext -> syntax_expand

appveyor: on_finish run after saving cache

Use after_test instead

travis: Windows build might be stuck as non-supported languages

See <https://docs.travis-ci.com/user/reference/windows/#supported-languages>
for list of Windows supported languages.

Auto merge of #4659 - lzutao:caching, r=phansch

Cache cargo binaries

changelog: none

Add `run-rustfix` to `inefficient_to_string`

Update adding_lints.md

Dogfood for `inefficient_to_string`

Add `inefficient_to_string` lint

Use CI caching and cargo feature install-upgrade

* do not force to install rustfmt
* use cargo-cache
* disable bash trace
* clone single branch

Auto merge of #4669 - lzutao:minimal-rustup, r=phansch

build: use rustup minimal profile to speed up Rust installation time

changelog: none

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

Rustup to rustc 1.40.0-nightly (237d54ff6 2019-10-15)

changelog: none

build: use rustup minimal profile to speed up Rust installation time

Rustup to rustc 1.40.0-nightly (237d54ff6 2019-10-15)

Auto merge of #4668 - lzutao:update-rtim, r=flip1995

 build: switch back to latest official RTIM release

Closes #4666

changelog: none

Auto merge of #4671 - flip1995:ice-4671, r=phansch

Fix ICE in `use_self` lint

The ICE is produced by building this span:
https://github.com/rust-lang/rust-clippy/blob/55e7818a06c8d83bead9c81e10e73ba33fb20890/clippy_lints/src/use_self.rs#L55-L60

`span` can start in the file the macro is defined in and end where the macro is called.

changelog: Fix ICE in `use_self` lint

Enable integration tests again

futures-rs, hyper, rls

Fix ICE in use_self lint

Add regression test for ICE in use-self lint

travis: break out before_script stage to test setup-toolchain work

build: do not trace deploy step

Early exit if there is any errors

build: switch back to latest official RTIM release

Auto merge of #4619 - james9909:unused-self, r=flip1995

Add a lint for unused self

changelog: Adds a new lint: `unused_self`

Closes #4550.

Change terminology from static to associated

Switch to OnlyBodies

Change lint to be pedantic

Remove prints

Add a new lint for unused self

Auto merge of #4560 - rust-lang:must-use-pure, r=phansch

new lints around`#[must_use]`

changelog: Add `must_use_candidate` lint,  add `must-use-unit` lint, add `double_must_use` lint

The first one checks if an public function or method has no mutable argument and mutates no non-local data and lints if it has no `#[must_use]` attribute. It will skip inner functions, because those are usually highly local and the attribute doesn't have as much benefit there.

The second lints `#[must_use]` attributes on functions and methods that return unit. Those attributes are likely a remnant from a refactoring that removed the return value.

The last one lints for `#[must_use]` attributrs without text on functions that return a type which is already marked `#[must_use]`. This has no auto-suggestion, because while it would be easy to do, there may be value in writing a detailed text for the attribute instead.

This fixes #4526

Auto merge of #4663 - lzutao:rustup, r=phansch

Rustup

changelog: none

Suppresses #4662.

r? @phansch

new lints around `#[must_use]` fns

`must_use_unit` lints unit-returning functions with a `#[must_use]`
attribute, suggesting to remove it.

`double_must_use` lints functions with a plain `#[must_use]`
attribute, but which return a type which is already `#[must_use]`,
so the attribute has no benefit.

`must_use_candidate` is a pedantic lint that lints functions and
methods that return some non-unit type that is not already
`#[must_use]` and suggests to add the annotation.

rustc-dev has been temporarily removed

Use my RTIM fork

Bump compiletest

Auto merge of #4649 - Lythenas:use_match_function_call, r=phansch

Use match_function_call wherever possible

Move `match_function_call` to `utils` and use it wherever possible as discussed in #4635.

changelog: none

Auto merge of #4654 - lzutao:rustc-dev, r=phansch

Rustup for rustc-dev

cc https://github.com/rust-lang/rust/pull/64823
Closes #4651
changelog: none