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.

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

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

use git version

Rustup for rustc-dev

Use match_function_call wherever possible

Move match_function_call to utils

Auto merge of #4646 - lzutao:make-pr-guides-visible, r=flip1995

PR template might not be stripped by contributors

cc d97fbdbb42c28ec9e051b23138d7898bae6836c4

So I think it would be better to make the template visible.
Also I renamed the template with extension `.md`.

changelog: none

PR template might not be stripped by contributors

Auto merge of #4616 - mati865:license, r=Manishearth

Fix license in Cargo.toml files

Missed in https://github.com/rust-lang/rust-clippy/pull/3269

changelog: none

r? @Manishearth

Auto merge of #4615 - nikofil:suspicious_unary_op_formatting, r=flip1995

New lint: suspicious_unary_op_formatting

fixes #4228

changelog: New lint: [`suspicious_unary_op_formatting`]

New lint: suspicious_unary_op_formatting

Lints when, on the RHS of a BinOp, there is a UnOp without a space
before the operator but with a space after (e.g. foo >- 1).

Signed-off-by: Nikos Filippakis <nikolaos.filippakis@cern.ch>

Auto merge of #4635 - Lythenas:suggestions-for-assert-false, r=flip1995

Add assert message to suggestion in lint assertions_on_constants

<!--
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.

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

[lint_naming]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints

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

- [x] suggest replacing `assert!(false, "msg")` with `panic!("msg")`
- [x] extend to allow ~~variables~~ any expression for `"msg"`
- ~~suggest replacing `assert!(false, "msg {}", "arg")` with `panic!("msg {}", "arg")`~~

changelog: add assert message to suggestion in lint assertions_on_constants

Work towards fixing: #3575

Auto merge of #4602 - EthanTheMaster:issue-4001, r=flip1995

Add suggestion for mul_add

Issue #4001: Whenever `a*b+c` is found where `a`,`b`, and `c` are floats, a lint is suggested saying to use `a.mul_add(b, c)`. Using `mul_add` may give a performance boost depending on the target architecture and also has higher numerical accuracy as there is no round off when doing `a*b`.

changelog: New lint: `manual_mul_add`

Addresses Issue #4001

Fixed typo

Fixes lint name and uses appropriate linting suggestion

changed lint help message

Added autofixable test

Added Autofixable Test

Removed Broken Autofixable File

updated lints

Generated Autofixable/Nonfixable Test Cases

Changed Suggestion Applicability

Updated Lint Count

Auto merge of #4641 - sinkuu:revert_workaround, r=phansch

Revert "Workaround cargo bug on Windows"

[Cargo is fixed on rust master](https://github.com/rust-lang/rust/pull/65186). This reverts PR #4624.

Fixes #4638

changelog: none

Auto merge of #4627 - kerhong:fix-typo-in-inherent-to-string, r=phansch

Fix typo in inherent_to_string documentation

A simple typo fix in `inherent_to_string` and `inherent_to_string_shadow_display` documentation

changelog: none

Auto merge of #4611 - rust-lang:doc-visibility, r=flip1995

account for doc visibility

This fixes #4608.

Also I noticed that the lint failed to look at trait and impl items. There's a small bit of fallout in the code, too, but not enough to warrant its own commit.

changelog: check docs of trait items and impl items, also make `missing_safety_doc` account for visibility

Auto merge of #4595 - rust-lang:rustbot, r=flip1995

Allow everyone to assign labels with rustbot

Also allows people to claim issues even if they aren't part of the org

changelog: none

Auto merge of #4592 - rust-lang:transmute-collection, r=flip1995

New lint: `unsound_collection_transmute`

changelog: Add `unsound_collection_transmute` lint

This fixes #4515

Revert "Workaround cargo bug on Windows"

This reverts commit 248251b3b26c5f21e2c2f5bfa5d85cc17b13fc05.
This reverts commit 20b73514395bb442fb16ce913b6c2b3965ff3156.

Auto merge of #4628 - flip1995:rustup, r=phansch

Rustup to rust-lang/rust#64874

TODO:
- [x] replace `rvalue_promotable_map` in [1]
- [ ] ~~fix [2] according to this comment https://github.com/rust-lang/rust/pull/64874#issuecomment-536203626 this should be merged with `consume`, but I didn't figure out how to merge them, yet.~~
- [ ] ~~fix [3]; What to do with `LoanCause`?~~

[2]+[3] probably have to be resolved by a rewrite of the lint. https://github.com/rust-lang/rust-clippy/pull/4628#issuecomment-538574944

[1]
https://github.com/rust-lang/rust-clippy/blob/54bf4ffd626970e831bb80c037f804a3b3450835/clippy_lints/src/methods/mod.rs#L1292-L1299

[2]
https://github.com/rust-lang/rust-clippy/blob/54bf4ffd626970e831bb80c037f804a3b3450835/clippy_lints/src/escape.rs#L126

[3]
https://github.com/rust-lang/rust-clippy/blob/54bf4ffd626970e831bb80c037f804a3b3450835/clippy_lints/src/escape.rs#L166-L176

I could need some help with [1]. The purpose of this is to "don't lint for constant values". cc @matthewjasper

For now I see what I can do with [2].

changelog: Temporary break `boxed_local` lint.

Reduce duplication

Match any expr for panic message

Make if_chain more readable

Add message to replace assert!(false) help

Move is_argument check into mutate

Run util/dev fmt

Get rid of rvalue_promotable_map method call

Merge consume and consume_pat in escape analysis

FIXME: This doesn't work and probably needs a rewrite of the lint

See https://github.com/rust-lang/rust-clippy/pull/4628#issuecomment-538574944

Use new format of licenses

Co-Authored-By: lzutao <taolzu@gmail.com>

Fix needless_pass_by_value

This also accidentally improved the spans of the suggestions

Rustup to rust-lang/rust#64874

Episode 1 - The simple cases

Auto merge of #4613 - Lythenas:lint-assert_eq-unit_exprs, r=flip1995

Add check for assert_eq macros to unit_cmp lint

changelog: Add check for unit comparisons through `assert_eq!`, `debug_assert_eq!`, `assert_ne!` and `debug_assert_ne!` macros to unit_cmp lint.

fixes #4481

Fix typo in inherent_to_string documentation

Auto merge of #4621 - JohnTitor:fix-windows, r=phansch

Use windows-sdk-10.1 to avoid installation failure

This fixes installation failure on Windows on Travis but we need to fix rustfmt issue first to pass the CI completely.

changelog: none

Auto merge of #4625 - phansch:rollup-qp7ki0h, r=phansch

Rollup of 2 pull requests

Successful merges:

 - #4509 (Fix false-positive of redundant_clone and move to clippy::perf)
 - #4614 (Allow casts from the result of `abs` to unsigned)

Failed merges:

changelog: none

r? @ghost

Auto merge of #4622 - Lythenas:fix-doc-formatting-for-mistyped-literal-suffixes, r=phansch

Correctly align doc of mistyped literal suffixes

changelog: Fix misaligned markdown list in doc of `mistyped_literal_suffixes`

Rollup merge of #4614 - HMPerson1:abs_cast_unsigned, r=flip1995

Allow casts from the result of `abs` to unsigned

changelog: Allow casts from the result of `abs` to unsigned in `cast_sign_loss`

Fixes #4605

Rollup merge of #4509 - sinkuu:redundant_clone_fix, r=llogiq

Fix false-positive of redundant_clone and move to clippy::perf

This PR introduces dataflow analysis to `redundant_clone` lint to filter out borrowed variables, which had been incorrectly detected.

Depends on https://github.com/rust-lang/rust/pull/64207.

changelog: Moved `redundant_clone` lint to `perf` group

# What this lint catches

## `clone`/`to_owned`

```rust
let s = String::new();
let t = s.clone();
```

```rust
// MIR
_1 = String::new();
_2 = &_1;
_3 = clone(_2); // (*)
```

We can turn this `clone` call into a move if

1. `_2` is the sole borrow of `_1` at the statement `(*)`
2. `_1` is not used hereafter

## `Deref` + type-specific `to_owned` method

```rust
let s = std::path::PathBuf::new();
let t = s.to_path_buf();
```

```rust
// MIR
_1 = PathBuf::new();
_2 = &1;
_3 = call deref(_2);
_4 = _3;                         // Copies borrow
StorageDead(_2);
_5 = Path::to_path_buf(_4); // (*)
```

We can turn this `to_path_buf` call into a move if

1. `_3` `_4` are the sole borrow of `_1` at `(*)`
2. `_1` is not used hereafter

# What this PR introduces

1. `MaybeStorageLive` that determines whether a local lives at a particular location
2. `PossibleBorrowerVisitor` that constructs [`TransitiveRelation`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc_data_structures/transitive_relation/struct.TransitiveRelation.html) of possible borrows, e.g. visiting `_2 = &1; _3 = &_2:` will result in `_3 -> _2 -> _1` relation. Then `_3` and `_2` will be counted as possible borrowers of `_1` in the sole-borrow analysis above.

Auto merge of #4624 - sinkuu:workaround_cargo, r=llogiq

Workaround cargo issue on appveyor

Use absolute paths for `cargo` and `rustfmt` to workaround https://github.com/rust-lang/cargo/issues/7475.

Appveyor passed on my fork: https://ci.appveyor.com/project/sinkuu/rust-clippy/builds/27870367

changelog: none

Use home::cargo_home

Workaround cargo bug on Windows

Correctly align doc of mistyped literal suffixes

Use windows-sdk-10.1 to avoid installation failure

Fix license in Cargo.toml files

Remove assert_ne example from doc

Update unit_cmp tests to include blocks for asserts

Mention asserts in doc for unit_cmp lint

Update the .stderr to include the backticks

Print assert macro name in backticks

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

extern rustc_index

Add comments

Resolve reviews

Apply suggestion

Co-Authored-By: ecstatic-morse <ecstaticmorse@gmail.com>

Test fixes

Add run-rustfix

Fix false-positive of redundant_clone and move to clippy::perf

Allow casts from the result of `abs` to unsigned

Add check for assert_eq macros to unit_cmp lint

New lint: unsound_collection_transmute

account for doc visibility