Merge branch 'master' into fix-2728

Auto merge of #3727 - phansch:rustup_unused_trim, r=matthiaskrgr

Rustup: unused trim result

Caused by https://github.com/rust-lang/rust/pull/57106

r? @matthiaskrgr

Rustup: unused trim result

Fix `cast_sign_loss` false positive

This checks if the value is a non-negative constant before linting about
losing the sign.

Because the `constant` function doesn't handle const functions, we check if
the value is from a call to a `max_value` function directly. A utility method
called `get_def_path` was added to make checking for the function paths
easier.

Fixes #2728

Auto merge of #3652 - Aehmlo:where_the_wild_things_are, r=phansch

Add wildcard_match_arm lint

This lint prevents using a wildcard in a match arm. Implemented as a restriction currently, because this is pretty much an edge case. See #3649 for more information.

Didn't add any tests because I wasn't sure how, but if someone wants to point me in the right direction, I'd be happy to!

wildcard_match_arm: Update lint count.

wildcard_match_arm: add nesting issue to known.

wildcard_match_arm: lint only enum matches.

wildcard_match_arm: update ui test stderr

wildcard_match_arm: format test.

wilcard_match_arm: run rustfmt.

wildcard_match_arm: add lint properly.

wildcard_match_arm: rename function.

We also don't need `ex` as an argument.

wildcard_match_arm: add simple ui test.

wildcard_match_arm: expand lint scope.

We're not only working with Results.

Change match_wild lint name to WILDCARD_MATCH_ARM.

Also fix message capitalization.

Add match_wild lint (#3649).

This lint prevents using a wildcard in a match.

Auto merge of #3648 - phansch:const_fn_lint, r=oli-obk

Add initial version of const_fn lint

This adds an initial version of a lint that can tell if a function could be `const`.

TODO:

- [x] Finish up the docs
- [x] Fix the ICE

cc #2440

Auto merge of #3716 - matthiaskrgr:sh, r=phansch

fetch_prs_between: add .sh file ending

r? @phansch

fetch_prs_between: add .sh file ending

cargo fmt

Auto merge of #3714 - mikerite:fix-2945, r=oli-obk

Fix `unit_arg` false positive

Ignore arguments with the question mark operator.

Closes #2945

Update various docs

* `const_transmute` currently also seems to depend on the `const_fn`
  feature.
* Only `Sized` is currently allowed as a bound, not Copy.

Use built-in entry_fn detection over self-built

cargo fmt

Reorganize conditionals: Run faster checks first

Maybe fix ICE?

Add initial version of const_fn lint

Fix `unit_arg` false positive

Ignore arguments with the question mark operator.

Closes #2945

Auto merge of #3700 - phansch:would_you_like_some_help_with_this_const_fn, r=oli-obk

Prevent incorrect cast_lossless suggestion in const_fn

`::from` is not a const fn, so applying the suggestion of
`cast_lossless` would fail to compile. The fix is to skip the lint if
the cast is found inside a const fn.

Fixes #3656

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

Rustup

fixes #3709

I'm currently in the process of making rustc's deprecation lint emit the `AtomicFoo::new(0)` suggestion

Rustfmt

Check hypothetically failing conversion

Remove tests for deprecated items

Update more changed iterator paths

Atomics constants are now handled by the deprecation lint

Update changed iterator paths

Update const slice processing

Auto merge of #3706 - robamler:patch-1, r=phansch

Fix documentation for `slow_vector_initialization`

This PR fixes the documentation for the lint `slow_vector_initialization`. The documentation recommended writing `vec![len; 0]` but the correct solution is `vec![0; len]`.

Auto merge of #3705 - matthiaskrgr:rustup, r=phansch

rustup

rustup https://github.com/rust-lang/rust/pull/57907/ and https://github.com/rust-lang/rust/pull/57726
Fixes #3708

update test stderr

run cargo fmt

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

for file in `fd \.rs$` ; do sed  -i  s/span_suggestion_with_applicability/span_suggestion/g $file ; done

for file in `fd \.rs$` ; do sed  -i  s/span_suggestion_short_with_applicability/span_suggestion_short/g $file ; done

for file in `fd \.rs$` ; do sed  -i  s/span_suggestions_with_applicability/span_suggestions/g $file ; done

Fix documentation for `slow_vector_initialization`

Change the recommended solution from `vec![len; 0]` to `vec![0; len]`.

Also fix grammar.

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

Auto merge of #3702 - phansch:changechangelog, r=matthiaskrgr

Remove unsafe_vector_initialization from added lints

It was deprecated before it reached a stable release,
no need to include it.

Remove unsafe_vector_initialization from added lints

It was deprecated before it reached a stable release,
no need to include it.

Auto merge of #3690 - mikerite:fix-3630-expect-fun-call, r=phansch

Fix `expect_fun_call` lint suggestions

This commit corrects some bad suggestions produced by the
`expect_fun_call` lint and enables `rust-fix` checking on the tests.

Addresses #3630

Prevent incorrect cast_lossless suggestion in const_fn

`::from` is not a const fn, so applying the suggestion of
`cast_lossless` would fail to compile. The fix is to skip the lint if
the cast is found inside a const fn.

Auto merge of #3701 - mikerite:fix-3118, r=phansch

Fix dogfood tests on Appveyor

This introduces a work-around for a bug in rustup.rs when excuting
cargo from a custom toolchain. Instead of trusting rustup to
invoke cargo from one of the release channels we just invoke
nightly cargo directly.

Incorporate review suggestions

Fix dogfood tests on Appveyor

This introduces a work-around for a bug in rustup.rs when excuting
cargo from a custom toolchain. Instead of trusting rustup to
invoke cargo from one of the release channels we just invoke
nightly cargo directly.

Auto merge of #3698 - sorin-davidoi:fix-common-metadata-no-deps, r=oli-obk

chore(cargo/dependencies/cargo-metadata): Upgrade to 0.7.1

Closes https://github.com/rust-lang/rust-clippy/issues/3692.

test(versioncheck): Use .no_deps()

test(versioncheck): Fix version equality check

chore(cargo/dependencies/cargo-metadata): Upgrade to 0.7.1

Closes https://github.com/rust-lang/rust-clippy/issues/3692.

Auto merge of #3699 - matthiaskrgr:itertools, r=oli-obk

dependencies: update itertools from 0.7 to 0.8

dependencies: update itertools from 0.7 to 0.8

Auto merge of #3691 - phansch:update_changelog, r=oli-obk

Update changelog with all changes since 0.0.212

This updates the changelog with all notable and user-facing changes since
0.0.212 has been released. I decided to group the changes by the Rust version
they are included in, as that's what makes the most sense to users.

Whenever a new rust version is released, we will have to go through the list
of commits and then update the changelog with the relevant changes.

[Rendered](https://github.com/rust-lang/rust-clippy/pull/3691/files?short_path=4ac32a7#diff-4ac32a78649ca5bdd8e0ba38b7006a1e)

Add script to fetch GitHub PRs between two commits

Auto merge of #3694 - phansch:fixed, r=phansch

gitattributes: Treat .fixed files as rust files

This way GitHub treats them as Rust code :crab: :eyes:

gitattributes: Treat .fixed files as rust files

Auto merge of #3693 - phansch:rustup, r=matthiaskrgr

Rustup

Due to https://github.com/rust-lang/rust/pull/51285

Rustup

Due to https://github.com/rust-lang/rust/pull/51285

Update changelog with all changes since 0.0.212

Fix `expect_fun_call` lint suggestions

This commit corrects some bad suggestions produced by the
`expect_fun_call` lint and enables `rust-fix` checking on the tests.

Addresses #3630

Auto merge of #3582 - Arkweid:add-lints-aseert-checks, r=flip1995

Add assert(true) and assert(false) lints

This PR add two lints on assert!(true) and assert!(false).
#3575

allow assertions_on_constants for collapsible_if and missing_test_files

Merge branch 'master' into add-lints-aseert-checks

* master: (58 commits)
  Rustfmt all the things
  Don't make decisions on values that don't represent the decision
  Improving comments.
  Rustup
  Added rustfix to the test.
  Improve span shortening.
  Added "make_return" and "blockify" convenience methods in Sugg and used them in "needless_bool".
  Actually check for constants.
  Fixed potential mistakes with nesting. Added tests.
  formatting fix
  Update clippy_lints/src/needless_bool.rs
  formatting fix
  Fixing typo in CONTRIBUTING.md
  Fix breakage due to rust-lang/rust#57651
  needless bool lint suggestion is wrapped in brackets if it is an "else" clause of an "if-else" statement
  Fix automatic suggestion on `use_self`.
  Remove negative integer literal checks.
  Fix `implicit_return` false positives.
  Run rustfmt
  Fixed breakage due to rust-lang/rust#57489
  ...

Auto merge of #3684 - g-bartoszek:sugg-snippet-modifications, r=phansch

"make_return" and "blockify" convenience methods, fixes #3683

…ed them in "needless_bool".

Auto merge of #3679 - daxpedda:use_self, r=phansch

Fix automatic suggestion on `use_self`.

In an example like this:
```rust
impl Example {
    fn fun_1() { }
    fn fun_2() {
        Example::fun_1();
    }
}
```
Clippy tries to replace `Example::fun_1` with `Self`, loosing `::fun_1` in the process, it should rather try to replace `Example` with `Self`.

**Question**
- There may be other paths that need the same treatment, but I'm not sure I understand them fully:
  - https://github.com/rust-lang/rust-clippy/blob/e648adf0866a1cea7db6ce2d33ea86e442f25377/clippy_lints/src/use_self.rs#L94-L96
  - https://github.com/rust-lang/rust-clippy/blob/e648adf0866a1cea7db6ce2d33ea86e442f25377/clippy_lints/src/use_self.rs#L225-L229

Auto merge of #3685 - rust-lang:rustup, r=flip1995

Rustup

Rustfmt all the things

Don't make decisions on values that don't represent the decision

Improving comments.

Rustup

Added rustfix to the test.

Improve span shortening.

Co-Authored-By: daxpedda <1645124+daxpedda@users.noreply.github.com>

Added "make_return" and "blockify" convenience methods in Sugg and used them in "needless_bool".

Auto merge of #3677 - daxpedda:integer_arithmetic, r=oli-obk

Remove negative integer literal checks.

Fixes #3678.

Actually check for constants.

Auto merge of #3676 - daxpedda:implicit_return, r=oli-obk

Fix `implicit_return` false positives.

Fixes the following false positives:
- linting on `if let` without `else` in a `loop` even with a present `return`
- linting on `unreachable!()`

Fixed potential mistakes with nesting. Added tests.

Auto merge of #3680 - g-bartoszek:needless-bool-else-if-brackets, r=oli-obk

needless bool lint suggestion is wrapped in brackets if it is an "els…

…e" clause of an "if-else" statement

formatting fix

Update clippy_lints/src/needless_bool.rs

Co-Authored-By: g-bartoszek <grzegorz.bartoszek@thaumatec.com>

formatting fix

Auto merge of #3681 - rmcteggart-r7:master, r=matthiaskrgr

Fixing typo in CONTRIBUTING.md

Noticed small typo in CONTRIBUTING readme when working on another issue

Auto merge of #3675 - mikerite:fix-build-20190120, r=matthiaskrgr

Fix build 20190120

Fixing typo in CONTRIBUTING.md

Fix breakage due to rust-lang/rust#57651

needless bool lint suggestion is wrapped in brackets if it is an "else" clause of an "if-else" statement

Fix automatic suggestion on `use_self`.

Remove negative integer literal checks.

Fix `implicit_return` false positives.

Run rustfmt

Fixed breakage due to rust-lang/rust#57489

Fix breakage due to rust-lang/rust#57755

Auto merge of #3674 - sinkuu:fmt_rustup, r=oli-obk

Catch up with `format_args` change

Catches up with a change in rust-lang/rust#57537. (Since the optimization is optional, this clippy PR can be merged before the rustc PR.)

Happened to fix a bug in `expect_fun_call`, that is the lint ignores more than
one arguments to `format`.

```
warning: use of `expect` followed by a function call
 --> src/main.rs:2:17
  |
2 |     Some("foo").expect(format!("{} {}", 1, 2).as_ref());
  |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try this: `unwrap_or_else(|| panic!("{} {}", 1))`
  |
```

Catch up with `format_args` change

Catches up with a change in rust-lang/rust#57537

Happened to fix a bug in `expect_fun_call`, that is the lint ignores more than
one arguments to `format`.