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.

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

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

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.

Auto merge of #4264 - flip1995:appveyor, r=flip1995

Disable AppVeyor builds on the master branch

AppVeyor is already checked on every merge of a PR, rechecking it
immediately after on the master branch is not necessary.

Resolves #4263

changelog: none

Disable AppVeyor builds on the master branch

AppVeyor is already checked on every merge of a PR, rechecking it
immediately after on the master branch is not necessary.

Auto merge of #4233 - thiagoarrais:jens1o-fix-1208, r=phansch,flip1995

Avoid reporting string_lit_as_bytes for long strings

Port of @jens1o code ([b76f939][jens1o_commit])

Fixes #1208

[jens1o_commit]: https://github.com/jens1o/rust-clippy/commit/b76f939ac2efcfe24900c286b3b7713d972d9088

<!--
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: bugfix for long strings as bytes

Auto merge of #4262 - bara86:master, r=flip1995

Use empty block instead of unit type for needless return

fixes #4238

changelog: Use empty block instead of unit type for needless return

Extract semantic constant

Use empty block instead of unit type for needless return

Avoid reporting string_lit_as_bytes for long strings

Port of @jens1o code ([b76f939][jens1o_commit])

Fixes #1208

[jens1o_commit]: https://github.com/jens1o/rust-clippy/commit/b76f939ac2efcfe24900c286b3b7713d972d9088

Co-authored-by: Thiago Arrais <thiago.arrais@gmail.com>

Add rustfmt nightly to appveyor install

Work around rustup fallback error on Windows

Update documentation to the dev fmt command

Remove format checks from CI scripts again.

Remove format checks from CI script

Improve fmt test failure message

Fix crash on `dev --limit-stderr-length`

Add dev fmt subcommand

Auto merge of #4258 - mikerite:fix-breakage-20190706, r=Manishearth

Fix breakage due to rust-lang/rust#61988

changelog: none

rustfmt

Address reviews

Fix breakage due to rust-lang/rust#61988

Auto merge of #4255 - mikerite:fix-warnings-20190706, r=Manishearth

Fix unknown lint warnings

changelog: none

Fix unknown lint warnings

Auto merge of #4254 - lzutao:hiridification-62168, r=Manishearth

Rustup HirIdification

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

changelog: none

Remove lint for ONCE_INIT

ONCE_INIT will be deprecated in rust 1.38.0

Update rustfix tests

Rename hir::map::local_def_id_from_hir_id to local_def_id

Auto merge of #4252 - mikerite:fix-unused-import-20190704, r=flip1995

Remove unused import

changelog: none

Remove unused import

Auto merge of #4251 - matthiaskrgr:rustup_10, r=Manishearth

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

changelog: none

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

Auto merge of #4250 - mikerite:fix-3992, r=phansch

Fix allow bug in `trivially_copy_pass_by_ref`

Closes #3992

changelog: Fix allow bug in `trivially_copy_pass_by_ref`

Auto merge of #4249 - matthiaskrgr:license_url, r=phansch

readme: fix urls in the license

The ">" at the end was detected as part of the url and caused it to 404.

changelog: none

Fix allow bug in `trivially_copy_pass_by_ref`

Closes #3992

readme: fix urls in the license

Auto merge of #4246 - mikerite:fix-4244, r=flip1995

Fix `match_same_arms` false negative

Closes #4244

changelog: Fix `match_same_arms` false negative

Fix `match_same_arms` false negative

Closes #4244

Auto merge of #4245 - flip1995:ice_4121, r=matthiaskrgr

Test for melted ICE #4121

resolves #4121

r? @matthiaskrgr

changelog: none

Reenable cargo and rls tests in travis

Test for melted ICE #4121