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

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

Fix dogfood test

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

Decrease maximum length for stderr files

Now at 275.

Improved imports

Auto merge of #4280 - phansch:uitests_matches, r=flip1995

UI Test Cleanup: Extract match_ref_pats tests

changelog: none

cc #2038

Auto merge of #4259 - Darth-Revan:origin/inherent_to_string, r=flip1995

Implement lint for inherent to_string() method.

Fixes #4247

changelog: Implement two new lints: `inherent_to_string` and `inherent_to_string_shadow_display`

1) Emits a warning if a type implements an inherent method `to_string(&self) -> String`
2) Emits an error if a type implements an inherent method `to_string(&self) -> String` and also implements the `Display` trait

Fix "unkown clippy lint" error in UI test.

Corrections for PR review.

Implement lint for inherent to_string() method.

Auto merge of #4278 - phansch:uitestcleanup_indexing, r=flip1995

UI Test Cleanup: Split out out_of_bounds_indexing

This moves the `out_of_bounds_indexing` lint tests to their own
directory.

changelog: none

cc #2038

UI Test Cleanup: Extract match_ref_pats tests

Update UI tests

Auto merge of #4276 - phansch:uitestcleanup, r=flip1995

UI Test Cleanup: Split up checked_unwrap tests

Let's slowly bring that ticket closer to the finish line :snail: :checkered_flag:

This splits up `tests/ui/checked_unwrap.rs` into:

 * `tests/ui/checked_unwrap/complex.rs`
 * `tests/ui/checked_unwrap/simple.rs`

Based on the naming of the methods in the tests.

changelog: none

cc #2038

Allow no_effect lint

for cleaner stderr file

Co-Authored-By: Philipp Krones <hello@philkrones.com>

Remove comment

Co-Authored-By: Philipp Krones <hello@philkrones.com>

cargo fmt

UI Test Cleanup: Split up checked_unwrap tests

This splits up `tests/ui/checked_unwrap.rs` into:

 * `tests/ui/checked_unwrap/complex.rs`
 * `tests/ui/checked_unwrap/simple.rs`

Based on the naming of the methods in the tests.

cc #2038

Removed lintining on never type.

Abstracted repeating strings into statics.

Auto merge of #4275 - jayhardee9:fix/cmp-signum, r=flip1995

Fix float_cmp false positive when comparing signum

fixes #4248

changelog: Fix float_cmp false positive when comparing signum

UI Test Cleanup: Split out out_of_bounds_indexing

This moves the `out_of_bounds_indexing` lint tests to their own
directory.

Auto merge of #4211 - flip1995:check_changelog, r=matthiaskrgr

Check for changelog entry in PR bodies

cc #4031 but now on the auto and try branches. (https://github.com/rust-lang/rust-clippy/issues/3955#issuecomment-502230021)

changelog: none

Lint

Fix float_cmp false positive when comparing signum

f1.signum() == f2.signum()
f1.signum() != f2.signum()
should not trigger a warning.

grep only lines, that start with 'changelog: '

Get PR number from bors commit message

Authenticate with github_api_token

Check PRs for changelog entry in the body

false positives fixes of `implicit_return`

-  Handle returning macro statements properly
-  Handle functions that return never type
-  Handle functions that panic but do not return never type

Auto merge of #4273 - mikerite:deny-warnings, r=flip1995

Deny warnings in CI

changelog: none

Deny warnings in CI

Auto merge of #3418 - phansch:add_travis_windows_build, r=me,flip1995

Fix Travis Windows build

Closes #3306

Disable dogfood on windows for faster build time on travis

Revert "Remove `CARGO_INCREMENTAL=0` from windows build"

This reverts commit 876a7e1f01cb14a63eee54cb44524a87161d5454.

Using incremental build on windows increases the build time on travis by
about 8 minutes.

Remove `CARGO_INCREMENTAL=0` from windows build

Test with different sysroots dependent on the OS

Build sys_root in driver with PathBuf instead of String

Don't re-set the LD_LIBRARY_PATH in base_tests.sh

Add master toolchain binaries to the PATH

Change conditional

Maybe uname == Linux was true on the windows VM?
This could be a way to avoid the secret environment variable issue with
Travis CI.

Stop allowing failures in Travis windows build

Auto merge of #4229 - euclio:lint-doc-generation-fix, r=flip1995

don't strip blank lines in lint documentation

changelog: don't strip blank lines in lint documentation

Fixes #4116.

This PR also switches the docs headings to deterministically display in the order that they are declared in the source, with "Configuration" always appearing last. It doesn't seem like there was a defined order before.

Auto merge of #4232 - mikerite:dev-fmt-4, r=flip1995

Add dev fmt subcommand

changelog: none

Auto merge of #4268 - mikerite:fix-4260, r=flip1995

Fix bug in `implicit_hasher` causing crashes

Skip linting if the type is from an external macro. Closes #4260.

changelog: Fix bug in `implicit_hasher` causing crashes

Merge branch 'master' into dev-fmt-4

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

Rustup `macro expansion and resolution`

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

changelog: none

Rustup `macro expansion and resolution`

Fix bug in `implicit_hasher` causing crashes

Skip linting if the type is from an external macro. Closes #4260.

Avoid rustfmt bug on Windows

Ignore generated fresh lifetimes in elision check.

Auto merge of #4257 - skade:improve-cast-alignment, r=flip1995

Improve cast_ptr_alignment lint

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

- [x] Followed [lint naming conventions][lint_naming]
- [x] Added passing UI tests (including committed `.stderr` file)
- [x] `cargo test` passes locally
- [x] Executed `util/dev update_lints`
- [x] Added lint documentation
- [x] 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 -->

* print alignment in bytes in the lint message
* ignore ZST left-hand types

Fixes #3797 and #4256
changelog:
* `cast_ptr_alignment`: Print alignment in bytes in the lint message
* `cast_ptr_alignment`: Ignore casting from ZST left-hand types

Improve cast_ptr_alignment lint

* print alignment in bytes in the lint message
* ignore ZST left-hand types

Auto merge of #4265 - vks:patch-1, r=flip1995

cast_ptr_alignment: Mention legal use under known problems

Refs #2881.

changelog: Mention know problems for cast_ptr_alignment

cast_ptr_alignment: Mention legal use under known problems

Refs #2881.