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

Use empty block instead of unit type for needless return

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

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

Fix warnings about unnecessary lifetime bounds

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

Auto merge of #4218 - lzutao:rustup, r=phansch

Rustup

changelog: none

Fix lifetime mismatch between LateContext and Ty

Fix missing field in ExprKind::Yield

Rustup rust-lang/rust#61941

Auto merge of #4210 - dtolnay:division, r=flip1995

Downgrade integer_division to restriction

I believe that this lint falls outside of the scope of opinionated pedantism of the other pedantic lints.

changelog: Downgrade integer_division lint from pedantic to restriction

Auto merge of #4217 - Manishearth:clog, r=flip1995

Update changelog headings

1.36 is beta now

changelog: none

Unify heading style

Make headings stable

Update changelog

Auto merge of #4216 - phansch:replace_nodeset, r=matthiaskrgr

Refactor: Replace NodeSet with HirIdSet

This saves us one HirId -> NodeId conversion.

changelog: none

Refactor: Replace NodeSet with HirIdSet

This saves us one HirId -> NodeId conversion.

Auto merge of #4215 - matthiaskrgr:rustup_9, r=flip1995

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

changelog: none

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

Auto merge of #4214 - d-dorazio:4204-fix-float-cmp-neq, r=flip1995

fix suggestion for floating point inequality

It should be of the form `(a - b).abs() > error` whereas it was `(a - b).abs() < error` that is instead what should be used for equality.

fixes #4204.

changelog: fix suggestion for floating point inequality

fix suggestion for floating points inequality

It should be of the form `(a - b).abs() > error` whereas it was
`(a - b).abs() < error` that is instead what should be used for equality.

Downgrade integer_division to restriction

Auto merge of #4209 - lzutao:TyCtxt-lifetime, r=Manishearth

Fix wrong lifetime of TyCtxt

Rustup rust-lang/rust#61817

changelog: none

Fix wrong lifetime of TyCtxt

Auto merge of #4206 - rust-lang:update_lints, r=phansch

Use replace_region_in_file for creating the lint list

r? @phansch

changelog: none

Use replace_region_in_file for creating the lint list

Auto merge of #4162 - krk:static-static, r=flip1995

Add lint for statics with explicit static lifetime.

changelog: Add lint for statics with explicit static lifetime, fixes #4138.

Restructure rename tests

Register rename to the LintStore.

Rename REDUNDANT_STATIC_LIFETIME to REDUNDANT_STATIC_LIFETIMES.

Remove pub from RedundantStaticLifetime.visit_type function.

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

Rename const_static_lifetime to redundant_static_lifetime.

Merge StaticConst and StaticStatic lints into StaticConst.

Use RedundantStaticLifetime in StaticStatic.

Move type-checking logic in StaticConst to RedundantStaticLifetime.

Add lint for statics with explicit static lifetime.

Auto merge of #4175 - yaahallo:master, r=oli-obk

Implement better help for clippy-driver

https://github.com/rust-lang/rust-clippy/issues/4173

Auto merge of #4201 - mgr-inz-rafal:typos_in_docs, r=matthiaskrgr

Typos and minor grammar corrections

Just some minor grammar issues and typos in documentation.

Auto merge of #4200 - lzutao:tyctxt-lifetime, r=matthiaskrgr

Fix wrong lifetime of TyCtxt

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

changelog: none

Fix wrong lifetime of TyCtxt

travis: Wait at most 30 minutes for base test

Sometimes travis gets slow and makes the builds spurious failure because
of no output in 10 minutes. This commit increase Travis timing-out time
at most 30 minutes before terminating the build.

minor fix