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

Auto merge of #4243 - mikerite:fix-4058, r=flip1995

Fix `never_loop` false positive

Closes #4058

changelog: none

Auto merge of #4222 - jfrikker:try_err, r=flip1995

Adding try_err lint

changelog: Adds the "try_err" lint, which catches instances of the following: Err(x)?
fixes #4212

Fix `never_loop` false positive

Closes #4058

Running rustfmt on test

Auto merge of #4240 - alex:uninit-intrinsic-prep, r=Manishearth

Removed use of intrinsics::uninit from tests.

This is in preperation for https://github.com/rust-lang/rust/pull/62150

Also allows deprecations in preperations for intrinsics::init being deprecated, which is a planned future step.

changelog: none

Removed use of intrinsics::uninit from tests.

This is in preperation for https://github.com/rust-lang/rust/pull/62150

Also allows deprecations in preperations for intrinsics::init being deprecated, which is a planned future step.

Auto merge of #4235 - Manishearth:rustup, r=Manishearth

Rustup to 5f9c0448dde167128d668da4555879f64e56af1d

Fixes fallout from https://github.com/rust-lang/rust/pull/62091/files

changelog: none

fmt

Remove a hir_to_node_id

find_by_hir_id -> find

get_parent_node_by_hir_id -> get_parent_node

Making try_err machine applicable

Auto merge of #4230 - flip1995:unsugar_if, r=Manishearth

Replace `unsugar_if` function with `is_if` function

cc https://github.com/rust-lang/rust-clippy/pull/4123#discussion_r296581719

changelog: none

r? @Manishearth

Simplify unsugar_if to just an if expr check

Remove ast:: prefix from types

Auto merge of #4123 - Centril:rustup-let-chains-ast, r=Manishearth

Fix fallout from rust-lang/rust PR 60861

Fixes incoming breakage for unlanded https://github.com/rust-lang/rust/pull/60861.

Tests are passing locally; the Rust PR now needs to land first.

@Manishearth also says we'll want to split out to a `collapsible_if_let` once we have let-chains working in Rust nightly or something.

Put 'if let' back into comment.

Pacify rustfmt.

Fix dogfood test failures.

PR comments

don't strip blank lines in lint documentation

Account for let_chains in collapsible_if ui test cases.

Fix fallout from rust-lang/rust PR 60861.

Adding try_err lint

Auto merge of #4220 - d-dorazio:4182-needless-return-void-functions, r=flip1995

make needless_return work with void functions

fixes https://github.com/rust-lang/rust-clippy/issues/4181.

changelog: make needless_return work with void functions.

I don't think the failure is related to my changes, but I'm not sure :thinking:

Auto merge of #4227 - lzutao:node-pruning, r=flip1995

Fix fallout cause NodeId pruning

Rustup rust-lang/rust#61984

changelog: none

Fix fallout cause NodeId pruning

cargo fmt

Auto merge of #4225 - jeremystucki:refactoring, r=flip1995

Remove needless lifetimes

changelog: none

more idiomatic code

Auto merge of #3919 - matthiaskrgr:readme_3, r=flip1995

readme: update

Remove the crates.io badge from the readme.
Installing clippy via crates.io is deprecated and rustup should be used instead, thus we should not promote crates.io here.

Don't have clippy::correctness" lint category in bold letters, I think it looks a little off.

Remove another unnecessary lifetime

Remove needless lifetimes

update tests and fix lints in clippy

make needless_return work with void functions

Auto merge of #4223 - mikerite:fix-breakage-2019-06-21, r=flip1995

Fix breakage due to rust-lang/rust#61968

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

Remove unnecssary lifetime from trait_ref_of_method

Fix breakage due to rust-lang/rust#61968

readme: update

Remove the crates.io badge from the readme.
Installing clippy via crates.io is deprecated and rustup should be used instead, thus we should not promote crates.io here.

Don't have the entire clippy::correctness line in bold letters, I think it looks a little off.

Auto merge of #4221 - lzutao:redundant-lifetime, r=matthiaskrgr

Fix warnings about unnecessary lifetime bounds

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

changelog: none