Auto merge of #3725 - mikerite:fix-2728, r=phansch

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

Document `get_def_path`

Merge pull request #3735 from jsgf/setup-toolchain

Add setup-toolchain.sh script to configure the master version of rustc,

Add setup-toolchain.sh script to configure the master version of rustc,
and update CONTRIBUTING.md accordingly.

Auto merge of #3737 - phansch:2018, r=phansch

Transition leftover test libs to Rust 2018

To tick off some checkboxes in https://github.com/rust-lang/rust/issues/58099 =)

Transition leftover test libs to Rust 2018

Auto merge of #3723 - rhysd:issue3721, r=oli-obk

Implement dbg_macro rule

Fixes  #3721

This patch adds new `dbg_macro` rule to check `dbg!` macro use.

Since this is my first patch to clippy, I'm not confident about following points:

- ~~Currently only checks `dbg!` span. Is it possible to check if the `dbg!` macro is provided by standard library or user-defined? If it's possible, I can make the check more strict.~~ Resolved as https://github.com/rust-lang/rust-clippy/pull/3723#discussion_r252417934
- ~~Is category `style` correct for this rule?~~'restriction' is used instead
- ~~Should I use `span_lint_and_sugg` instead of `span_lint`? Currently entire message is put as `msg`.  But later part `ensure to avoid having uses of it in version control` may be put as suggestion.~~ Done
- I'm not native English speaker. The message and doc may not be natural as English.

run `util/dev update_lints` and `cargo fmt --all`

Merge branch 'master' into issue3721

prefer `if` to `match`

use snippet for making a suggestion if possible

Auto merge of #3736 - mikerite:fix-build-20190203, r=phansch

Fix breakage due to rust-lang/rust#58079

The rustc change added HirId to a few nodes. As I understand it, the plan is
to remove the NodeId from these nodes eventually. Where the NodeId was
not being matched, I used `..` to try and avoid further breakage. Where it
was, I used `_` to make the fix easier when NodeId is removed.

Fix breakage due to rust-lang/rust#58079

The rustc change added HirId to a few nodes. As I understand it, the plan is
to remove the NodeId from these nodes eventually. Where the NodeId was
not being matched, I used `..` to try and avoid further breakage. Where it
was, I used `_` to make the fix easier when NodeId is removed.

Auto merge of #2857 - avborhanian:master, r=phansch

Adding lint test for excessive LOC.

This is a WIP for #2377. Just wanted to pull in because I had a few questions:

1. Is it okay that I'm approaching this via counting by looking at each line in the snippet instead of looking at the AST tree? If there's another way to do it, I want to make sure I'm doing the correct way, but I wasn't sure since the output AST JSON doesn't seem to contain whitespace.

2. My function is definitely going to trigger the lint, so also wanted to see if there was something obvious I could do to reduce it.

3. Are the two tests fine, or is there something obvious I'm missing?

4. Obviously bigger question - am I approaching the line count correctly. Current strategy is count a line if it contains some code, so skip if it's just comments or empty.

Auto merge of #3731 - phansch:travis, r=phansch

Travis: Don't run integration tests on every PR commit

This does not save Clippy any time but it makes sure that the concurrent
build limit is not reached as quickly for the `rust-lang` Travis
account.

I can't create a permalink to the discussion somehow, so here's an
excerpt from the Infra channel:

```
[11:57 PM] pietroalbini: there is a clippy build (20 jobs) and a packed_simd one (42 builders)  and a rustc one which isn't scheduling atm
[11:58 PM] pietroalbini: I don't think there is a way to prioritize rustc builds in the queue on travis, right?
[12:22 AM] alexcrichton: pietro: I don't think so no
[12:22 AM] alexcrichton: If it's a problem we should cull builds on other projects
[12:22 AM] alexcrichton: The rust repo is the #1 priority
```

Since the integration tests are rarely failing these days, I think it's
fine to not run them on every commit.

If needed, it's also still possible to do a complete test run with `@bors try`.

Auto merge of #3732 - phansch:fix_ice_3720, r=oli-obk

Fix ICE in vec_box lint and add run-rustfix

Includes https://github.com/rust-lang/rust-clippy/pull/3726

`hir::Ty` doesn't seem to know anything about type bounds and
`cx.tcx.type_of(def_id)` caused an ICE when it was passed a generic type
with a bound:

```
src/librustc_typeck/collect.rs:1311: unexpected non-type Node::GenericParam: Type { default: None, synthetic: None }
```

Converting it to a proper `Ty` fixes the ICE and catches a few more places
where the lint applies.

Fixes #3720

add more test cases for dbg_macro rule

Update clippy_lints/src/types.rs

Co-Authored-By: avborhanian <avborhanian@gmail.com>

Update clippy_lints/src/types.rs

Co-Authored-By: avborhanian <avborhanian@gmail.com>

use span_help_and_lint() instead of span_lint_and_sugg()

Updating code to ignore rustfmt issue.

Updated readme.

Changing single character string to a character match.

Fix test broken by removing comment.

Updating to just warn for one test.

Adding back tests, but also reducing threshold by 1

rustfmt

Moving tests to ui-toml to make use of clippy.toml

Updating number of lines for the failing test to be > 100.

Due to updating the configuration to be 101 instead of 51

Adding lint for too many lines.

Skipping check if in external macro.

Fix test broken by removing comment.

Updating to just warn for one test.

Adding back tests, but also reducing threshold by 1

rustfmt

Moving tests to ui-toml to make use of clippy.toml

Reworking function logic, and adding doc example.

This should fix line count logic issues that the previous code had, with assumptions it would make.

Running util/dev to update README/CHANGELOG

Updating number of lines for the failing test to be > 100.

Due to updating the configuration to be 101 instead of 51

Adding lint for too many lines.

Remove conditionals from base builds

We _always_ want to execute these, also on the master branch.

Make vec_box MachineApplicable

Fix ICE in vec_box lint and add run-rustfix

`hir::Ty` doesn't seem to know anything about type bounds and
`cx.tcx.type_of(def_id)` caused an ICE when it was passed a generic type
with a bound:

```
src/librustc_typeck/collect.rs:1311: unexpected non-type Node::GenericParam: Type { default: None, synthetic: None }
```

Converting it to a proper `Ty` fixes the ICE and catches a few more
places where the lint applies.

Auto merge of #3726 - phansch:some_renaming, r=oli-obk

Some renamings: s/ast_ty/hir_ty and s/StructField/hir::StructField

I think in both cases the new names make the code more understandable.
For `StructField` specifically because there's one in
[`syntax::ast`][ast] and one in [`rustc::hir`][hir].

[ast]: https://doc.rust-lang.org/nightly/nightly-rustc/syntax/ast/struct.StructField.html
[hir]: https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/struct.StructField.html

Travis: Don't run integration tests on every PR commit

This does not save Clippy any time but it makes sure that the concurrent
build limit is not reached as quickly for the `rust-lang` Travis
account.

I can't create a permalink to the discussion somehow, so here's an
excerpt from the Infra channel:

```
[11:57 PM] pietroalbini: there is a clippy build (20 jobs) and a packed_simd one (42 builders)  and a rustc one which isn't scheduling atm
[11:58 PM] pietroalbini: I don't think there is a way to prioritize rustc builds in the queue on travis, right?
[12:22 AM] alexcrichton: pietro: I don't think so no
[12:22 AM] alexcrichton: If it's a problem we should cull builds on other projects
[12:22 AM] alexcrichton: The rust repo is the #1 priority
```

Since the integration tests are rarely failing these days, I think it's
fine to not run them on every commit.

If needed, it's also still possible to do a `try` build with `@bors try`.

Move `max_value` handling to consts module

Merge branch 'master' into fix-2728

`cargo +nightly fmt` at clippy_lints/

remove TODO comment which was already done

add dbg_macro rule to CHANGELOG.md and update count in README

fix category and use suggestion

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

Some renamings: s/ast_ty/hir_ty and s/StructField/hir::StructField

I think in both cases the new names make the code more understandable.
For `StructField` specifically because there's one in
[`syntax::ast`][ast] and one in [`rustc::hir`][hir].

[ast]: https://doc.rust-lang.org/nightly/nightly-rustc/syntax/ast/struct.StructField.html
[hir]: https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/struct.StructField.html

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

implement dbg_macro rule (fixes #3721)

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