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

Respond to review comments

Update README and CHANGELOG using the util scripts, refine the help message and fix the float_cmp error.

Auto merge of #4301 - flip1995:dummy, r=Manishearth

Bump version of clippy_dummy

The crates.io page of clippy still suggest to install `clippy-preview`
instead of `clippy` I think it's time to change this.

Thanks to the Stuttgart Meetup for discovering this!

r? @Manishearth could you also publish this after merging?

changelog: none

Auto merge of #4302 - matthiaskrgr:rustup_13, r=phansch

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

changelog: none

ci: temporarily disable rustfmt checks/tetss since it's broken for nightly

Updated test stderr

Added doc comment fixed type printout

Added a doc comment for the lint and fixed the printout of the type so it prints T not type(T)

Respond to comments and improve printout

Now get the trait names for the diagnostic message and removed more
`let c: fn(_) -> _ = T; hashes from hir_utils

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

Bump version of clippy_dummy

The crates.io page of clippy still suggest to install `clippy-preview`
instead of `clippy` I think it's time to change this.

Thanks to the Stuttgart Meetup for discovering this!

Responded to comments and fixed compile bug

Removed the hash of `let c: fn(_,_) -> _ = ExprKind::Cast` and
fixed compile issue by collecting HirVec into an actual Vec

Fixed more compile errors

Moved to rustc::hir::Ty

Fix some of the compile errors

Changed Ty to ty, added lifetime 'tcx

Lint for type repetition in trait bounds.

This lint adds warning if types are redundantly repeated in trait bounds i.e. `T: Copy, T: Clone` instead of `T: Copy + Clone`. This is a late pass trait lint and has necessitated the addition of code to allow hashing of TyKinds without taking into account Span information.

Auto merge of #4297 - matthiaskrgr:rustup_12, r=matthiaskrgr

rustup

changelog: none

update test stderr, not sure which rustc pull request caused this.

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

Auto merge of #4266 - uHOOCCOOHu:fix/async_fn_lifetime, r=flip1995

Ignore generated fresh lifetimes in elision check

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

fixes #3988

changelog: Ignore generated fresh lifetimes in elision check.

**HELP**: It seems `tests/ui` are compiled under edition 2015, and I don't know how to add tests for this properly.

Here is the test input it had already passed:
```rust
#![feature(async_await)]
#![allow(dead_code)]

async fn sink1<'a>(_: &'a str) {} // lint
async fn sink1_elided(_: &str) {} // ok

async fn one_to_one<'a>(s: &'a str) -> &'a str { s } // lint
async fn one_to_one_elided(s: &str) -> &str { s } // ok
async fn all_to_one<'a>(a: &'a str, _b: &'a str) -> &'a str { a } // ok
// async fn unrelated(_: &str, _: &str) {} // Not allowed in async fn

// #3988
struct Foo;
impl Foo {
    pub async fn foo(&mut self) {} // ok
}

// rust-lang/rust#61115
async fn print(s: &str) { // ok
    println!("{}", s);
}

fn main() {}

```

Fix tests for edition 2018 compatibility

Revert "Revert global fmt config and use `rustfmt::skip`"

This reverts commit b70b3b14aa3125fe4f74a6a71fb7a7b7e78ab13f.

Auto merge of #4292 - mikerite:fix-breakage-20190721, r=matthiaskrgr

Fix breakage due to rust-lang/rust#62705

Also rename `OUTER_EXPN_INFO` to `OUTER_EXPN_EXPN_INFO` to match new
function names.

changelog: none

Fix breakage due to rust-lang/rust#60913

Fix breakage due to rust-lang/rust#62705

Also rename `OUTER_EXPN_INFO` to `OUTER_EXPN_EXPN_INFO` to match new
function names.

Revert global fmt config and use `rustfmt::skip`

Fix fmt

Auto merge of #4287 - matthiaskrgr:rustup_11, r=phansch

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

changelog: none

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

Auto merge of #4286 - phansch:update_pulldown_cmark3, r=flip1995

Update pulldown-cmark to 0.5.3

Fixes a couple of crashes of which I added one example to our tests.

changelog: Update `pulldown-cmark` to 0.5.3 to fix potential crashes in `doc_markdown` lint

Auto merge of #4285 - matthiaskrgr:rustup_backticks, r=phansch

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

(was merged as part of https://github.com/rust-lang/rust/pull/62782 )

changelog: none

Update pulldown-cmark to 0.5.3

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

(was merged as part of https://github.com/rust-lang/rust/pull/62782 )

Auto merge of #4282 - phansch:decrease_stderr_maximum_length, r=flip1995

Decrease maximum length for stderr files

Now at 275.

changelog: none

cc #2038

Add test

Auto merge of #4274 - daxpedda:implicit_return_fixes, r=flip1995

false positives fixes of `implicit_return`

-  Handle returning macro statements properly (remove "this error originates in a macro outside of the current crate")
-  Handle functions that return never type
-  Handle functions that panic but do not return never type

changelog: Fix false positives in `implicit_return` lint pertaining to macros and panics

Auto merge of #4284 - flip1995:rustfmt, r=phansch

Format code

changelog: none

Format code