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.

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

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

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

Auto merge of #4599 - lzutao:zero-ptr-suggestion, r=flip1995

Add suggestion for zero-ptr lint

changelog: Improve suggestion of `zero_ptr` lint

Auto merge of #4603 - rust-lang:needless-doc-main, r=flip1995

New lint: needless_doc_main

changelog: Add `needless_doc_main` lint

Add suggestion for zero-ptr lint

New lint: needless_doc_main

Auto merge of #4590 - flip1995:ice_4579, r=Manishearth

Fix ICE #4579

Fixes #4579
Fixes #4584

r? @phansch

changelog: Fix ICE caused by Clippys const-utils

Disable hyper and futures-rs integration tests

Fix ICE #4579

Add regression test for ICE #4579

Merge pull request #4606 from Manishearth/rustup

Fix some tests

Remove tests that only ICE on CI

Allow const_err in out_of_bounds_indexing tests

Merge pull request #4604 from Manishearth/rustup

Rustup to rustc 1.40.0-nightly (702b45e40 2019-10-01)

Use new spans for expansion checking in loop lints

Auto merge of #4601 - lzutao:clean-up-unused-vars, r=phansch

Clean up some unused vars

changelog: none

Clean up some unused vars

Auto merge of #4600 - lzutao:rustup-63492, r=oli-obk

Rustup rust-lang/rust#63492

changelog: none

Rustup rust-lang/rust#63492

Auto merge of #4593 - james9909:fix-multiple-inherent-impls, r=llogiq

Fix false positive in `multiple_inherent_impl`

changelog: Fix false positive in `multiple_inherent_impl` by ignoring impls derived from macros.

Closes #4578.

Allow everyone to assign labels with rustbot

Auto merge of #4594 - matthiaskrgr:rustup_18, r=phansch

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

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

changelog: none

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

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

Ignore impls derived from macros

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

Rustup to rust-lang/rust#64813

cc rust-lang/rust#64843

changelog: none

Remove clippy::author attribute from trailing_zeroes test

Move author issue test to author subdir

Fix author lint

Rustup to rust-lang/rust#64813

Auto merge of #4589 - mikerite:booleans-refactor-20190925, r=phansch

Refactor `booleans`

Remove unused output from `suggest(..)`

changelog: none

Refactor `booleans`

Remove unused output from `suggest(..)`

Auto merge of #4585 - michaelsproul:arith-assign-op, r=llogiq

Detect mutating arithmetic in integer_arithmetic restriction lint

changelog: detect mutating arithmetic (like +=) in `integer_arithmetic` restriction lint

Detect assignment ops in integer_arithmetic

Auto merge of #4582 - matthiaskrgr:rustup_17, r=Manishearth

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

changelog: none

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

changelog: none

Auto merge of #4581 - Manishearth:osx-enable, r=matthiaskrgr

Re enable OSX builders

Fixes #4576

changelog: none

Re enable OSX builders

Fixes #4576

Auto merge of #4580 - lzutao:rustup, r=flip1995

Rustup rust-lang/rust#64513

changelog: none

Remove unused import

Rustup https://github.com/rust-lang/rust/pull/64513

Auto merge of #4568 - mikerite:fix-4548, r=flip1995

Fix `nonminimal-bool` false positive

Closes #4548
Closes #3847

changelog: Fix `nonminimal-bool` false positive

Auto merge of #4569 - james9909:add-comparison-chain, r=oli-obk

Add a new lint for comparison chains

changelog: Adds a new lint: `comparison_chain`.

`comparison_chain` lints all `if` conditional chains where all the conditions are binary comparisons on the same two operands and will suggest a rewrite with `match`.

Closes #4531.

Auto merge of #4570 - lzutao:call_site_toplevel_ref_arg, r=phansch

Fix macro expansion in toplevel_ref_arg lint

changelog: Fix macro expansion in toplevel_ref_arg lint

Remove unused attribute in test

Fix macro expansion in toplevel_ref_arg lint

Add regression test for macro expansion

Auto merge of #4575 - Manishearth:suggestions, r=oli-obk

Make more tests rustfixable

Fixes https://github.com/rust-lang/rust-clippy/issues/3630

changelog: Improve suggestions for many lints in preparation for `cargo fix --clippy`

r? @phansch @yaahc

Downgrade op_ref to a MaybeIncorrect suggestion

unnecessary_operation: make test rustfixable

unnecessary_clone: split rustfixable lint out into separate test

string_add, string_add_assign: split tests, make one rustfixable

redundant_static_lifetimes: split test, make rustfixable

renamed_builtin_attr: make test rustfixable

redundant_pattern_matching: make rustfixable

redundant_closure_call: split tests into fixable

map_unit_fn: make test rustfixable

map_unit_fn: fix applicability

option_map_unit_fn: Split into fixable/unfixable

map_unit_fn: rename tests to fixable

non_copy_const: remove incorrect suggestion

needless_return: add allow()s to test, make rustfixable