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

Add example to needless_range_loop

Auto merge of #4160 - lzutao:bump-cargo_metadata, r=matthiaskrgr

Bump cargo_metadata to 0.8.0

changelog: none

Auto merge of #4159 - lzutao:into-outer_expn_info, r=matthiaskrgr

Make use of new `outer_expn_info()` function

Find by simple command: `git grep -w expn_info`.

changelog: none

Closes #4151

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

Closes #4144

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

Bump cargo_metadata to 0.8.0

Make use of new `outer_expn_info()` function

Auto merge of #4158 - phansch:rustup_tests, r=Manishearth

Fix ellipsis_inclusive_range_patterns lint warnings

Changed from `allow` to `warn` in https://github.com/rust-lang/rust/pull/61342

changelog: none

Fix ellipsis_inclusive_range_patterns lint warnings

Changed from `allow` to `warn` in https://github.com/rust-lang/rust/pull/61342

add suggestions for print/write with newline lint

Auto merge of #4157 - phansch:update_pulldown_cmark2, r=matthiaskrgr

Update pulldown_cmark to 0.5.2

Includes various parsing fixes. Most notably [this PR][pr]

changelog: none

[pr]: https://github.com/raphlinus/pulldown-cmark/pull/325

Auto merge of #4113 - rust-lang:author, r=flip1995

Fix `Block`  dump in author lint

changelog: The `#[clippy::author]` attribute now emits correct pattern code for block expressions

Update pulldown_cmark to 0.5.2

Includes various parsing fixes. Most notably [this PR][pr]

[pr]: https://github.com/raphlinus/pulldown-cmark/pull/325

Auto merge of #4155 - phansch:rustup_trait_obj, r=oli-obk

Rustup to https://github.com/rust-lang/rust/pull/61203

See https://github.com/rust-lang/rust/pull/61203

Migrates all trait objects to use `dyn` in our ui tests

changelog: none

Rustup to https://github.com/rust-lang/rust/pull/61203

Migrates all trait objects to use `dyn`

Auto merge of #4153 - matthiaskrgr:rustup_5, r=flip1995

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

changelog: none

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

Auto merge of #4152 - matthiaskrgr:rustup_4, r=matthiaskrgr

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

(which is included in https://github.com/rust-lang/rust/pull/61274)

changelog: none

rustup https://github.com/rust-lang/rust/pull/61164
(which is included in https://github.com/rust-lang/rust/pull/61274)

Don't rustfmt author test

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

Fix `Block`  dump in author lint

Auto merge of #4142 - agnxy:rename-assoc, r=flip1995

Rename "Associated*" to "Assoc*"

This is to fix the breakage introduced by rust-lang/rust#60163.

changelog: none

Auto merge of #4071 - matthiaskrgr:sizeof, r=flip1995

trivially_copy_pass_by_ref: print size of type and limit in the lint message

changelog: trivially_copy_pass_by_ref: print size of type and limit in the lint message

Auto merge of #4150 - rust-lang:rustup, r=oli-obk

Rustup to rustc 1.36.0-nightly (fa40a111f 2019-05-27)

changelog: none

Rustup to rustc 1.36.0-nightly (fa40a111f 2019-05-27)

Auto merge of #4137 - euclio:let-return, r=oli-obk

tweak `let_and_return` diagnostic

changelog: `let_and_return`: label the unnecessary let binding and convert the note to a structured
suggestion

Auto merge of #4149 - flip1995:rollup-6knpdqb, r=flip1995

Rollup of 2 pull requests

Successful merges:

 - #4102 (Fix match_same_arms to fail late)
 - #4119 (Improve non ascii literal)

Failed merges:

r? @ghost

Rollup merge of #4119 - BO41:non_ascii_literal, r=flip1995

Improve non ascii literal

This PR improves the example of the [non_ascii_literal](https://rust-lang.github.io/rust-clippy/master/index.html#non_ascii_literal) lint.
It also makes it auto-fixable.

Please review. This is my first PR to this project.
(Thanks @flip1995 for the help :)

changelog: none
fixes #4117

Rollup merge of #4102 - Urriel:fix/4096_match_same_arms, r=flip1995

Fix match_same_arms to fail late

Changes:
- Add a function search_same_list which return a list of matched expressions
- Change the match_same_arms implementation behavior. It will lint each same arms found.

fixes #4096

changelog: none

Auto merge of #3832 - HarrisonMc555:use_last, r=flip1995

Implement "Use last" lint

Closes #3673

This lint checks the use of `x.get(x.len() - 1)` and suggests `x.last()` (where `x` is a `Vec`).

There's at least one more thing that needs to happen here. I believe I am correctly checking most of the scenarios and avoiding false positives. However, if different `Vec`s are used for the `x.get` and `y.len`, then it will emit a false positive. In other words, I need to check to make sure the `Vec`s are the same.

Also, a lot of these commits were temporary and not helpful to the project history...am I supposed to squash on merge? If so, how do I do that?

changelog: New lint: `get_last_with_len`

Auto merge of #4148 - sanxiyn:typo, r=Manishearth

Fix typo in comment

Fix typo in comment

Auto merge of #4145 - rust-lang:rustup, r=oli-obk

Rustup to rustc 1.36.0-nightly (2268d9923 2019-05-26)

changelog: none

Rustfmt

Rustup to rustc 1.36.0-nightly (2268d9923 2019-05-26)

Rename "Associated*" to "Assoc*"

This is to fix the breakage introduced by rust-lang/rust#60163.

Auto merge of #4141 - phansch:rustup5, r=oli-obk

Rustup to https://github.com/rust-lang/rust/pull/61026

None

Rustup to https://github.com/rust-lang/rust/pull/61026