Re-add false positive check

Add raw string regression test for useless_format lint

Re-factor useless_format lint

Update Unicode lint tests

Auto merge of #4368 - RalfJung:invalid_ref, r=oli-obk

deprecate invalid_ref lint

This fixes the `invalid_ref` lint test to no longer fail when https://github.com/rust-lang/rust/pull/63346/ lands.  I also fixed the lint itself, because its wording made no sense: there is no "reference to zeroed/uninitialized memory" here.

changelog: none

update_lints

deprecate invalid_ref lint

Auto merge of #4362 - lzutao:expect-on-cstring_as_ptr, r=flip1995

Fix lint_cstring_as_ptr for expect

Closes #4312
changelog: none

Auto merge of #4361 - lzutao:fix-raw-string-on-single_char_pattern, r=phansch

Fix lint_single_char_pattern on raw string literal

Closes #4356
changelog: Handle raw string literal on `single_char_literal` lint.

Add more tests for raw string literal

Auto merge of #4360 - atouchet:patch-1, r=phansch

Fix HTTPS in links

I accidentally missed these in my previous PR.

changelog: none

Fix lint_cstring_as_ptr for expect

Fix lint_single_char_pattern on raw string literal

Fix HTTPS in links

Auto merge of #4358 - lzutao:cleanup_implicit_return, r=flip1995

Use source_callsite() instead of recursive outer_expn_info()

changelog: none

Auto merge of #4357 - lzutao:update-pr-template, r=flip1995

template: Update how to format source code with ./util/dev

changelog: none

Auto merge of #4355 - lzutao:macro_expn_try_err, r=flip1995

Fix macro expansion in try_err lint

Fixes #4309

changelog: none

template: Update how to format source code with ./util/dev

Use source_callsite() instead of recursive outer_expn_info()

Use snippet_with_macro_callsite suggested by flip1995

Fix macro expansion in try_err lint

Auto merge of #4350 - matthiaskrgr:readme_cargo_fix_clippy, r=phansch

README: mention experimental cargo fix clippy support

mention that cargo fix has experimental support for applying some clippy lint suggestions via "cargo fix -Z unstable-options --clippy"

[Rendered](https://github.com/matthiaskrgr/rust-clippy/tree/readme_cargo_fix_clippy#auto-applying-clippy-suggestions)

changelog: none

Update README.md (fix typo)

Co-Authored-By: Philipp Hansch <dev@phansch.net>

README: mention that cargo fix has experimental support for applying some clippy lint suggestions via "cargo fix -Z unstable-options --clippy"

Auto merge of #4339 - phansch:rustfix_needless_bool, r=flip1995

Add run-rustfix for needless_bool lint

This splits up the needless_bool tests into `fixable.rs` and
`simple.rs`. `simple.rs` contains the code that triggers the lint
diagnostic without a suggestion.

changelog: none

cc #3630

Auto merge of #4345 - phansch:enum_variants_fix, r=flip1995

Don't emit enum_variant_names if remainder starts with a numeric

changelog: Fix false positive in `pub_enum_variant_names` and `enum_variant_names`

As [per the reference](https://doc.rust-lang.org/reference/identifiers.html), identifiers must start with a letter. So we don't suggest a better
variant naming in case the remainder would start with a numeric.

Fixes #739

Auto merge of #4346 - atouchet:links, r=flip1995

Update Readme

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

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

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

changelog: none

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

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

Unblock https://github.com/rust-lang/rust/pull/63280

changelog: none

Update Readme

Don't emit enum_variant_names if remainder starts with a numeric

As [per the reference](https://doc.rust-lang.org/reference/identifiers.html),
identifiers must start with a letter. So we don't suggest a better
variant naming in these cases.

Fixes #739

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

Add run-rustfix for needless_bool lint

This splits up the needless_bool tests into `fixable.rs` and
`simple.rs`. `simple.rs` contains the code that triggers the lint
diagnostic without a suggestion.

Auto merge of #4335 - phansch:fix_needless_bool_suggestion, r=flip1995

Fix needless_bool suggestion with if--else-if--else

changelog: Fix `needless_bool` suggestion with if--else-if--else

Closes #4334

fmt

Fix needless_bool suggestion with if-else-if-else

Closes #4334

Auto merge of #4203 - Urriel:feat/333_calling_main, r=flip1995

Add recursion check on main function

Changes:
- Add MainRecursion lint to Clippy
- Check for no-std setup

fixes #333

changelog: Add `main_recursion` lint

Ignore no_std-test on windows and macos

Run update_lints and fmt

Move expression check to LateLintPass

Changes:
- Move from EarlyLintPass
- Fix entrypoint check with function path def_id.

Add recursion check on main function

Changes:
- Add MainRecursion lint to clippy
- Check for no-std setup

fixes #333

Auto merge of #4338 - flip1995:rollup-9cm4jbr, r=flip1995

Rollup of 4 pull requests

Successful merges:

 - #4329 (Doctests: Enable running doc tests for pedantic lints)
 - #4330 (Doctests: Enable running doc tests for nursery lints)
 - #4331 (Doctests: Enable running doc tests for restriction lints)
 - #4332 (Split up cast.rs tests, run-rustfix for unnecessary_cast)

Failed merges:

r? @ghost

changelog: none

Rollup merge of #4332 - phansch:rustfix_unnecessary_cast, r=flip1995

Split up cast.rs tests, run-rustfix for unnecessary_cast

This splits up the cast.rs tests and enables rustfix tests for the part
of the `unnecessary_cast` lint that emits `MachineApplicable`
suggestions.

changelog: none

cc #3630

Rollup merge of #4331 - phansch:doctests_restriction, r=flip1995

Doctests: Enable running doc tests for restriction lints

changelog: Enabled remaining doc tests for lint documentation page

master: 202 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out
this PR: 231 passed; 0 failed; 123 ignored; 0 measured; 0 filtered out

Closes #4319 (assuming this is merged after #4329 and #4330)

Rollup merge of #4330 - phansch:doctests_nursery, r=flip1995

Doctests: Enable running doc tests for nursery lints

changelog: none

master: 202 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out
this PR: 213 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out

cc #4319

Rollup merge of #4329 - phansch:doctests_pedantic, r=flip1995

Doctests: Enable running doc tests for pedantic lints

changelog: none

master: 202 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out
this PR: 254 passed; 0 failed; 131 ignored; 0 measured; 0 filtered out

cc #4319

Auto merge of #4337 - flip1995:let_unit, r=phansch

 Make let_unit lint suggestion auto applicable

changelog: Make `let_unit` lint suggestion auto applicable

Auto merge of #4333 - phansch:rustfix_decimal_literal_representation, r=flip1995

Add run-rustfix for decimal_literal_representation lint

changelog: none

cc #3630

Add tests for auto fixable let_unit suggestion

Make let_unit lint suggestion auto applicable

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

Rustup to https://github.com/rust-lang/rust/pull/63213

changelog: none

Rustup to https://github.com/rust-lang/rust/pull/63213

Add run-rustfix for decimal_literal_representation lint

cc #3630

Split up cast.rs tests, run-rustfix for unnecessary_cast

This splits up the cast.rs tests and enables rustfix tests for the part
of the `unnecessary_cast` lint that emits `MachineApplicable`
suggestions.

cc #3630

Doctests: Enable running doc tests for restriction lints

Doctests: Enable running doc tests for nursery lints

Doctests: Enable running doc tests for pedantic lints

Auto merge of #4327 - phansch:doctests_perf, r=flip1995

Doctests: Enable running doc tests for perf lints

changelog: none

This should be possible to merge independently of #4325

cc #4319

Auto merge of #4325 - phansch:doctests_complexity, r=flip1995

Doctests: Enable running doc tests for complexity lints

changelog: none

master: `113 passed; 0 failed; 91 ignored; 0 measured; 0 filtered out`
this PR: `181 passed; 0 failed; 110 ignored; 0 measured; 0 filtered out`

cc #4319

Auto merge of #4328 - phansch:rustup, r=phansch

Rustup to latest rustc master

Broken due to:

* https://github.com/rust-lang/rust/pull/63180 (`Existential` -> `OpaqueTy`)
* https://github.com/rust-lang/rust/pull/63121 (New fields for `FormatSpec`)

changelog: none

Ignore missing rustfmt

Rustup to latest rustc master

Broken due to:

* https://github.com/rust-lang/rust/pull/63180 (`Existential` -> `OpaqueTy`)
* https://github.com/rust-lang/rust/pull/63121 (New fields for `FormatSpec`)

Also enable doc tests for internal lints

Because there were fixed in the parent commit.

Doctests: Enable running doc tests for perf lints

This should be possible to merge independently of #4325

cc #4319

Doctests: Fix all complexity lint docs

cc #4319

Auto merge of #4322 - Y0hy0h:patch-1, r=phansch

Improve documentation on implicit_hasher lint

Provide an example of how to fix the implicit_hasher lint.

Fixes #3475.

changelog: none

Fix missing import

Auto merge of #4314 - chansuke:add-negation-to-is_empty, r=flip1995

Add negation to `len_zero` lint to show more explicit message.

Fixes #4304 I have updated the `len_zero` to show the required negation in case of like the below case.

```
fn main() {
    let v = vec![1];
    if v.len() > 0 {
    }
}
```

changelog: Clarify suggestion of `len_zero` lint.

Update `stderr` file

Auto merge of #4307 - flip1995:unnecessary_unwrap, r=oli-obk

Move {unnecessary,panicking}_unwrap out of nursery

Resolves #2437

changelog: Move `{unnnecessary,panicking}_unwrap` out of nursery

Fix doc tests

Remove Known problems for UNNECESSARY_UNWRAP

This shouldn't happen with NLL

Move UNNECESSARY_UNWRAP to complexity and PANICKING_UNWRAP to correctness

Auto merge of #4318 - izik1:typo, r=phansch

Fix `for on` typo

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

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

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

closes #4317

changelog: minor typo fix

Improve documentation on implicit_hasher lint

Provide an example of how to fix the lint.

Auto merge of #4316 - phansch:method_rs_cleanup2, r=flip1995

UI Test Cleanup: No wrong_self_convention in methods.rs

These cases are already covered in `tests/ui/wrong_self_convention.rs`.

cc #2038

changelog: none

Auto merge of #4315 - phansch:fn_to_numeric_cast_applicability, r=flip1995

Mark `fn_to_numeric_cast` lints as MaybeIncorrect

At least for now so that `cargo fix --clippy` is not causing problems
with this lint. See #3896 for the remaining problems with the suggestions of this lint.

changelog: none

cc #3630, #3896

Auto merge of #4303 - mikerite:dev-fmt-20190728, r=flip1995

Make fmt test and command more robust

changelog: none

Fix `for on` typo

Update formatting

Merge branch 'master' into dev-fmt-20190728

UI Test Cleanup: No wrong_self_convention in methods.rs

These cases are already covered in `tests/ui/wrong_self_convention.rs`.

cc #2038

Mark `fn_to_numeric_cast` lints as MaybeIncorrect

At least for now so that `cargo fix --clippy` is not causing problems
with this lint.

cc #3630, #3896

Auto merge of #4313 - Manishearth:owl, r=yaahallo

Don't nudge people towards toilet closures when producing owl results

`.map_err(drop)` should never be linted since sometimes you want to produce `Result<(), ()>` and the alternative is `.map_err(|_| ())`, which can be ugly. We don't seem to, but it's good to specifically test for this.

changelog: none

r? @yaahallo

Don't nudge people towards toilet closures when producing owl results

Add required negation in non_zero lint

Auto merge of #4308 - jakubadamw:use-self-variant-scope, r=flip1995

Extend the `use_self` lint to suggest uses of `Self::Variant`

Fixes #4296.

changelog: Extend the `use_self` lint to suggest uses of `Self::Variant`

Update all the code to pass the updated `use_self` lint.

One struct required a temporary `#[allow(dead_code)]` annotation due to
a bug in the Rust compiler: https://github.com/rust-lang/rust/issues/63151.

Extend the `use_self` lint to suggest uses of `Self::Variant`.

Auto merge of #3766 - xd009642:issue-3764, r=flip1995

trait bounds lint - repeated types

This PR is to tackle https://github.com/rust-lang/rust-clippy/issues/3764 it's still a WIP and doesn't work but this is an initial stab. It builds though I haven't added any tests as I'm not sure where lint tests should go?

Unfortunately, it seems id isn't tied to the type itself but I guess where it is in the AST? Looking at https://manishearth.github.io/rust-internals-docs/syntax/ast/struct.Ty.html I can't see any members that would let me tell if a type was repeated in multiple trait bounds.

There may be other issues with how I've implemented this so any assistance is appreciated!

changelog: Add new lint: `type_repetition_in_bounds`

Auto merge of #4306 - mikerite:fix-breakage-20190730, r=flip1995

Fix breakage due to rust-lang/rust#61856

changelog: none

Fix breakage due to rust-lang/rust#61856

Correct typo

Make appveyor build ignore rustfmt unavailability

Fix dogfood test

Enable rustfmt tests

This reverts commit d73a953db7a8120ec00b31c57d7092aff0f2c10b.

Skip fmt test if rustfmt is unavailble

dev-fmt: better error handling

Check if rustfmt is installed at the start and exit if it isn't.

Hash discriminant of lifetime.name

Hash discriminant of Lifetime::Name

Updated tests.

Removed unnecessary type repetition in float test and regenerated stderr
Regenerated type_repetition stderr