PR comments

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

make it pass dogfood

run rustfmt

fix padding and put clippy someplaces

show default lint levels

group printing

prelim arg parse

move Lint static def into its own module

switch to sorted usable lints

Update clippy_dev/src/main.rs

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

initial commit for help improvements on clippy-driver

Typos and minor grammar corrections

Auto merge of #4195 - thiagoarrais:division-of-integer-literals, r=flip1995

Adds lint for integer division

Hi, folks! This is my first contribution to clippy and my first real piece of Rust code.

This is supposed to add a lint that warns for division of integers (#109). Please let me know if you need any changes.

Fixes #109

changelog: Add lint for integer division

Adds lint for integer division

Auto merge of #4191 - g-bartoszek:redundant-closure-deref, r=flip1995

redundant_closure_for_method_calls fixes

lint does not trigger when there is a difference in mutability
lint does not trigger when the method belongs to a trait which is not implemebted directly (Deref)

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

redundant_closure_for_method_calls fixes:
lint does not trigger when there is a difference in mutability
lint does not trigger when the method belongs to a trait which is not implemebted directly (Deref)

Auto merge of #4199 - flip1995:implicit_return_docs, r=flip1995

Fix implicit_return docs

Resolves #4197

changelog: none

Auto merge of #4198 - matthiaskrgr:rustup_8, r=flip1995

rustup

changelog: none

cc @lzutao

r? @phansch

Fix implicit_return docs

rustup https://github.com/rust-lang/rust/pull/61758/files

Remove wrong lifetime from LintContext

Auto merge of #4190 - projedi:fix-eta, r=flip1995

Fixing eta with respect to lazy evaluation.

This fixes #4187

changelog: `redundant_closure`: stop linting on expressions returning a function, which is then directly used by the closure

Workaround for rust-lang/rustfmt#3615

Fixing eta with respect to lazy evaluation.

Auto merge of #4188 - matthiaskrgr:rustup_7, r=Manishearth

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

changelog: none

travis: disable rls integration test.

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

Auto merge of #4185 - mikerite:outer_expn_info, r=Manishearth

Add OUTER_EXPN_INFO lint

changelog: none

Add OUTER_EXPN_INFO lint

Auto merge of #4177 - mikerite:compiler_lint_functions_fewer_strings, r=phansch

Remove `to_string()`s from CompilerLintFunctions

changelog: none

Auto merge of #4136 - euclio:println-writeln-suggestions, r=flip1995

add suggestions for print/write with newline lint

changelog: Add machine-applicable suggestions for `print!`/`write!` with newline lints.

Auto merge of #4178 - matthiaskrgr:rustup_6, r=oli-obk

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

changelog: none

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

Remove `to_string()`s from CompilerLintFunctions

Auto merge of #4164 - mikerite:fix-4144, r=mikerite

Fix .map(..).unwrap_or_else(..) bad suggestion

Closes #4144

Fix comment grammar

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

Auto merge of #4168 - lzutao:fix-script, r=flip1995

Do not install rustup-toolchain if it is in PATH

I find it quiet annoying because I manually build `rustup-toolchain-install-master`
and install it in `PATH` other than in `~/.cargo/bin`. So everytime I run the script,
it always reinstall `rustup-toolchain-install-master` for me.

changelog: none

Auto merge of #4165 - BO41:needless_range_loop, r=phansch

Add example to needless_range_loop

adds a "could be written as" example

btw, is it correct that the lint triggers even if the index is used not just for getting the values by index?
So that I have to add `.iter().enumerate()` to still get an index?

changelog: none

Auto merge of #4163 - lzutao:doc-fix-foo-example, r=phansch

doc: Remove `pub` from trait definition in foo lint example

changelog: none

Auto merge of #4169 - lzutao:bump-cargo_metadata, r=Manishearth

Bump cargo metadata

Sorry, I forgot to bump `cargo_metadata` in sub-crates.

changelog: none

r? @matthiaskrgr

Bump clap in clippy_dev

Do not install rustup-toolchain if it is in PATH

Bump cargo_metadata of clippy_lints

Change from ignore to rust

Auto merge of #4166 - mati865:rustup, r=Manishearth

Rustup for https://github.com/rust-lang/rust/pull/61276

changelog: none

Cargo fmt

Rustup for https://github.com/rust-lang/rust/pull/61276