Auto merge of #4403 - xd009642:type_repetition_improvements, r=phansch,flip1995

Improvements to `type_repetition_in_bounds`

Improvements to the `type_repetition_in_bounds` trait based on feedback from #4380 #4326 #4323

Currently just make it pedantic. Hopefully, more to come

changelog: move `type_repetition_in_bounds` to `pedantic`

Auto merge of #4404 - BO41:patch-1, r=flip1995

Add "could be written as" example to MANUAL_MEMCPY

<!--
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 `./util/dev fmt`

[lint_naming]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints

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

Run update_lints

Auto merge of #4394 - jeremystucki:suspicious_map, r=flip1995

Implement "suspicious_map" lint

Resolves #4010

changelog: New lint `suspicious_map`.

Change lint type to 'complexity'

Merge branch 'origin/master' into suspicious_map

Auto merge of #4407 - mikerite:fix-breakage-20190818, r=flip1995

Fix breakage due to  rust-lang/rust#61708

Includes commits from #4406

changelog: none

Fix breakage due to rust-lang/rust#61708

Fix assertions_on_constants lint

Make code cleaner and more readable

Add "could be written as" example to MANUAL_MEMCPY

Made lint pedantic

Auto merge of #4397 - kraai:remove-a, r=flip1995

Remove "a" from single_match_else description

changelog: none

Auto merge of #4392 - phansch:cleanups, r=flip1995

Remove unused dev scripts/files

781794d4: Remove util/update_lints.py

 * This has been deprecated for ~10 months, so I think it's fine to remove now.

90a42937: Remove publishing things

 * Clippy is no longer published via crates.io, so these docs and the script
   are not needed anymore.

changelog: none

Auto merge of #4398 - kraai:outer_expn_data, r=flip1995

Use outer_expn_data instead of outer_expn_info

`outer_expn_info` has been removed, so use its replacement, `outer_expn_data`, instead.

changelog: none

Run rustfmt

Use outer_expn_data instead of outer_expn_info

Remove "a" from single_match_else description

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

Rustup "Remove `Spanned` from `{ast,hir}::FieldPat`"

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

changelog: none

Rustup "Remove `Spanned` from `{ast,hir}::FieldPat`"

Add ui test

Implement lint 'suspicious_map'

Remove util/update_lints.py

This has been deprecated for ~10 months, so I think it's fine to remove now.

Remove publishing things

Clippy is no longer published via crates.io, so these docs and the
script are not needed anymore.

Auto merge of #4391 - flip1995:toml_ver, r=phansch

Also specify patch version of toml crate

cc rust-lang/rust#63587

The patch update of the toml crate to version 0.5.3 recently broke some tests: #4378. For rustc, we have to define the complete version though.

changelog: none

Also specify patch version of toml crate

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