Auto merge of #4382 - jeremystucki:unnecessary_fold_span, r=flip1995

Change span of unnecessary_fold lint

Resolves #4381

changelog: Change linted span of `unnecessary_fold`

Auto merge of #4348 - phansch:deprecate-unused-unused-collect, r=flip1995

Deprecate unused_collect lint

I found this because we only had two test cases in total for this lint.
It turns out the functionality is fully covered by rustc these days.

[Playground Examples](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=eb8ee6db389c77180c9fb152d3c608f4)

changelog: Deprecate `unused_collect` lint. This is fully covered by rustc's `#[must_use]` on `collect`

cc #2846

Auto merge of #4389 - flip1995:allow_win_failures, r=phansch

Allow windows failures on travis

The windows build breaks about every second travis run. Let's disable it, until we got a fix.

changelog: none

Allow failures on the travis windows build again

Auto merge of #4388 - flip1995:rustup, r=phansch

Rustup

Supersedes #4387 and #4385

This removes tests with the `try!` macro in them completely. There is no need for Clippy to support the `try!` macro, since it is deprecated now.

[`StmtKind`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/enum.StmtKind.html) got a new variant [`Semi`](https://doc.rust-lang.org/nightly/nightly-rustc/rustc/hir/enum.StmtKind.html#variant.Semi), which Just Works with the `author` lint. Nice.

changelog: none

Rustup to rust-lang/rust#62984

Lint redundant_semicolon was added to rustc

Rustup to rust-lang/rust#62672

try macro is deprecated now, so Clippy will drop the support for it also

Rustup to rust-lang/rust#63543

Deprecate unused_collect lint

I found this because we only had two test cases in total for this lint.
It turns out the functionality is fully covered by rustc these days.

[Playground Examples](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=eb8ee6db389c77180c9fb152d3c608f4)

changelog: Deprecate `unused_collect` lint. This is fully covered by
rustc's `#[must_use]` on `collect`

cc #2846

Auto merge of #4383 - phansch:update_changelog, r=Manishearth

Update changelog for Rust 1.37

Also updates the rest of the changelog wrt. to the latest releases and commit hashes.

changelog: none

[Rendered](https://github.com/phansch/rust-clippy/blob/update_changelog/CHANGELOG.md#rust-137)

Fix links to lints in changelog

Remove unused import

Update changelog for Rust 1.37

Also updates the rest of the changelog wrt. to the latest releases.

Update tests

Use different span

Add multiline test

Auto merge of #4379 - matthiaskrgr:url_dep, r=phansch

update url dependency from 1.7.0 to 2.1.0

changelog: none

Auto merge of #4363 - phansch:fix_lint_deprecation, r=flip1995

Update lint deprecation for tool lints

changelog: Allow tool lints (`clippy::*`) to be deprecated

Our lint deprecation previously didn't work for tool lints, because
`register_removed` was registering lints to be removed _without_ the
`clippy` prefix.

Fixes #4349

update url dependency from 1.7.0 to 2.1.0

Auto merge of #4369 - mikerite:fix-4293, r=flip1995

 Fix `wrong_self_convention` issue

Resolves #4293

changelog: Fix `wrong_self_convention` issue

Auto merge of #4231 - jeremystucki:flat-map, r=flip1995

Implement flat_map lint

Fixes #4224

changelog: New Lint `flat_map_identity` to detect unnecessary calls to `flat_map`

Auto merge of #4378 - flip1995:ui-toml, r=flip1995

Update ui-toml tests

changelog: none

Update ui-toml tests

Merge branch 'origin/master' into flat-map

Update test

Use correct span

Add 'unnecessary_flat_map.fixed'

Run 'update_lints'

Use 'span_lint_and_sugg'

Minor refactoring

Add deprecation tests for deprecated tool lints

Move old lint deprecation tests to deprecated_old.rs

Fix invalid_ref deprecation

Keep old deprecated lints deprecated as non-tool, too

Update lint deprecation for tool lints

Our lint deprecation previously didn't work for tool lints, because
`register_removed` was registering lints to be removed _without_ the
`clippy` prefix.

Auto merge of #4373 - flip1995:filter_changelog_none, r=phansch

Filter out PRs with changelog: none

cc #4372

Filtering out the `changelog: none` merge commits, to unclutter the output.

changelog: none

Change lint type from 'pedantic' → 'complexity'

Filter out PRs with changelog: none

Run 'update_lints'

Auto merge of #4365 - lukas-code:async_new_ret_no_self, r=flip1995

new_ret_no_self: allow Self in inner type for impl Trait return types

Check the inner types of associated types of a trait when checking for Self in the return type of a `new` method. This means that the following will no longer warn:
```rust
trait Trait {
    type Inner;
}

struct S;

impl S {
    fn new() -> impl Trait<Inner = Option<Self>> {
        struct TraitImpl;

        impl Trait for TraitImpl {
            type Inner = Option<S>;
        }

        TraitImpl
    }
}
```
```rust
#![feature(async_await)]

struct Connection;

impl Connection {
    async fn new() -> Result<Self, ()> {
        Ok(S)
    }
}
```
closes #4359

changelog: fix `new_ret_no_self` lint for async `new` functions.

Auto merge of #4367 - flip1995:doc_edition_2018_tests, r=phansch

Document how to write tests requiring the 2018 edition

[Rendered](https://github.com/flip1995/rust-clippy/blob/doc_edition_2018_tests/doc/adding_lints.md#Edition-2018-tests)

cc #4365

changelog: none

Update rustdoc

Refactor if_chain

Co-authored-by: Philipp Krones <hello@philkrones.com>

Auto merge of #4370 - vallentin:master, r=flip1995

Fixed repeated word

changelog: none

Rename 'flat_map' → 'flat_map_identity'

Handle calls with 'std::convert::identity'

Merge branch 'origin/master' into flat-map

Fixed repeated word

Auto merge of #4368 - RalfJung:invalid_ref, r=oli-obk

deprecate invalid_ref lint

This fixes the `invalid_ref` lint test to no longer fail when https://github.com/rust-lang/rust/pull/63346/ lands.  I also fixed the lint itself, because its wording made no sense: there is no "reference to zeroed/uninitialized memory" here.

changelog: none

update_lints

deprecate invalid_ref lint

Fix `wrong_self_convention` issue

Resolves #4293

Simplify `wrong_self_convention` code

Use actual types instead of hir types.

Rename implitem to impl_item

`impl_item` is proper snake case and used everywhere else.

fixed tests (again) and added a test that should actually cause a warning for new_ret_no_self

removed unused imports #2

removed unused imports

upgrade test to rust 2018

Document how to write tests requiring the 2018 edition

new_ret_no_self: walk associated types in impl Trait return types

Auto merge of #4362 - lzutao:expect-on-cstring_as_ptr, r=flip1995

Fix lint_cstring_as_ptr for expect

Closes #4312
changelog: none

Auto merge of #4361 - lzutao:fix-raw-string-on-single_char_pattern, r=phansch

Fix lint_single_char_pattern on raw string literal

Closes #4356
changelog: Handle raw string literal on `single_char_literal` lint.

Add more tests for raw string literal

Auto merge of #4360 - atouchet:patch-1, r=phansch

Fix HTTPS in links

I accidentally missed these in my previous PR.

changelog: none

Fix lint_cstring_as_ptr for expect

Fix lint_single_char_pattern on raw string literal

Fix HTTPS in links

Auto merge of #4358 - lzutao:cleanup_implicit_return, r=flip1995

Use source_callsite() instead of recursive outer_expn_info()

changelog: none

Auto merge of #4357 - lzutao:update-pr-template, r=flip1995

template: Update how to format source code with ./util/dev

changelog: none

Auto merge of #4355 - lzutao:macro_expn_try_err, r=flip1995

Fix macro expansion in try_err lint

Fixes #4309

changelog: none

template: Update how to format source code with ./util/dev

Use source_callsite() instead of recursive outer_expn_info()

Use snippet_with_macro_callsite suggested by flip1995

Fix macro expansion in try_err lint

Auto merge of #4350 - matthiaskrgr:readme_cargo_fix_clippy, r=phansch

README: mention experimental cargo fix clippy support

mention that cargo fix has experimental support for applying some clippy lint suggestions via "cargo fix -Z unstable-options --clippy"

[Rendered](https://github.com/matthiaskrgr/rust-clippy/tree/readme_cargo_fix_clippy#auto-applying-clippy-suggestions)

changelog: none

Update README.md (fix typo)

Co-Authored-By: Philipp Hansch <dev@phansch.net>

README: mention that cargo fix has experimental support for applying some clippy lint suggestions via "cargo fix -Z unstable-options --clippy"

Auto merge of #4339 - phansch:rustfix_needless_bool, r=flip1995

Add run-rustfix for needless_bool lint

This splits up the needless_bool tests into `fixable.rs` and
`simple.rs`. `simple.rs` contains the code that triggers the lint
diagnostic without a suggestion.

changelog: none

cc #3630

Auto merge of #4345 - phansch:enum_variants_fix, r=flip1995

Don't emit enum_variant_names if remainder starts with a numeric

changelog: Fix false positive in `pub_enum_variant_names` and `enum_variant_names`

As [per the reference](https://doc.rust-lang.org/reference/identifiers.html), identifiers must start with a letter. So we don't suggest a better
variant naming in case the remainder would start with a numeric.

Fixes #739

Auto merge of #4346 - atouchet:links, r=flip1995

Update Readme

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

Auto merge of #4340 - lzutao:rustup, r=oli-obk

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

Unblock https://github.com/rust-lang/rust/pull/63280

changelog: none

Update Readme

Don't emit enum_variant_names if remainder starts with a numeric

As [per the reference](https://doc.rust-lang.org/reference/identifiers.html),
identifiers must start with a letter. So we don't suggest a better
variant naming in these cases.

Fixes #739

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

Add run-rustfix for needless_bool lint

This splits up the needless_bool tests into `fixable.rs` and
`simple.rs`. `simple.rs` contains the code that triggers the lint
diagnostic without a suggestion.

Auto merge of #4335 - phansch:fix_needless_bool_suggestion, r=flip1995

Fix needless_bool suggestion with if--else-if--else

changelog: Fix `needless_bool` suggestion with if--else-if--else

Closes #4334

fmt

Fix needless_bool suggestion with if-else-if-else

Closes #4334

Auto merge of #4203 - Urriel:feat/333_calling_main, r=flip1995

Add recursion check on main function

Changes:
- Add MainRecursion lint to Clippy
- Check for no-std setup

fixes #333

changelog: Add `main_recursion` lint

Ignore no_std-test on windows and macos

Run update_lints and fmt

Move expression check to LateLintPass

Changes:
- Move from EarlyLintPass
- Fix entrypoint check with function path def_id.

Add recursion check on main function

Changes:
- Add MainRecursion lint to clippy
- Check for no-std setup

fixes #333

Auto merge of #4338 - flip1995:rollup-9cm4jbr, r=flip1995

Rollup of 4 pull requests

Successful merges:

 - #4329 (Doctests: Enable running doc tests for pedantic lints)
 - #4330 (Doctests: Enable running doc tests for nursery lints)
 - #4331 (Doctests: Enable running doc tests for restriction lints)
 - #4332 (Split up cast.rs tests, run-rustfix for unnecessary_cast)

Failed merges:

r? @ghost

changelog: none

Rollup merge of #4332 - phansch:rustfix_unnecessary_cast, r=flip1995

Split up cast.rs tests, run-rustfix for unnecessary_cast

This splits up the cast.rs tests and enables rustfix tests for the part
of the `unnecessary_cast` lint that emits `MachineApplicable`
suggestions.

changelog: none

cc #3630

Rollup merge of #4331 - phansch:doctests_restriction, r=flip1995

Doctests: Enable running doc tests for restriction lints

changelog: Enabled remaining doc tests for lint documentation page

master: 202 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out
this PR: 231 passed; 0 failed; 123 ignored; 0 measured; 0 filtered out

Closes #4319 (assuming this is merged after #4329 and #4330)

Rollup merge of #4330 - phansch:doctests_nursery, r=flip1995

Doctests: Enable running doc tests for nursery lints

changelog: none

master: 202 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out
this PR: 213 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out

cc #4319

Rollup merge of #4329 - phansch:doctests_pedantic, r=flip1995

Doctests: Enable running doc tests for pedantic lints

changelog: none

master: 202 passed; 0 failed; 122 ignored; 0 measured; 0 filtered out
this PR: 254 passed; 0 failed; 131 ignored; 0 measured; 0 filtered out

cc #4319

Auto merge of #4337 - flip1995:let_unit, r=phansch

 Make let_unit lint suggestion auto applicable

changelog: Make `let_unit` lint suggestion auto applicable

Auto merge of #4333 - phansch:rustfix_decimal_literal_representation, r=flip1995

Add run-rustfix for decimal_literal_representation lint

changelog: none

cc #3630