Auto merge of #4409 - jeremystucki:export, r=phansch

[Doc] Ignore lines starting with #

Resolves #4353

changelog: none

Auto merge of #4438 - lzutao:author-litstr-missing, r=phansch

Add missing field to LitKind::Str

changelog: none

Add missing field to LitKind::Str

Auto merge of #4439 - lzutao:fix-format, r=phansch

Re-factor format lint

cc #4432

changelog: none

Run update_lints for Unicode lint

Re-add false positive check

Add raw string regression test for useless_format lint

Re-factor useless_format lint

Merge pull request #4440 from lzutao/fix-unicode

Update Unicode lint tests

Update Unicode lint tests

Auto merge of #4372 - phansch:changelog_update, r=flip1995

docs: Explain how to update the changelog

cc @Manishearth Is this more or less how you did it last time?

[Rendered](https://github.com/phansch/rust-clippy/blob/changelog_update/doc/changelog_update.md#changelog-update)

changelog: none

Auto merge of #4352 - phansch:fix_redundant_pattern_matching, r=flip1995

Fix some suggestions for redundant_pattern_matching

.. and change the Applicability to `MaybeIncorrect`.

Fixes the problem displayed in https://github.com/rust-lang/rust-clippy/issues/4344#issuecomment-519206388.

We now append `{}` to the suggestion so that the conditional has the
correct syntax again.

(If we were to _remove_ the `if` instead, it would trigger the
`unused_must_use` warning for `#[must_use]` types.)

changelog: Fix some suggestions for `redundant_pattern_matching`

Auto merge of #4425 - mikerite:4375, r=flip1995

Fix `temporary_cstring_as_ptr` false negative

Fixes #4375.

Changes the check to test when `.unwrap().as_ptr()` is called on any
`Result<CString, _>` as suggested by @flip1995
(https://github.com/rust-lang/rust-clippy/issues/4375#issuecomment-520724123).

changelog: Fix `temporary_cstring_as_ptr` false negative

Add two more tests, allow 2 other lints.

Fix `temporary_cstring_as_ptr` false negative

Fixes #4375.

Changes the check to test when `.unwrap().as_ptr()` is called on any
`Result<CString, _>` as suggested by @flip1995
(https://github.com/rust-lang/rust-clippy/issues/4375#issuecomment-520724123).

Add more testcases for redundant_pattern_matching

These should make sure that, when the suggestions are fixed, they are
fixed for all these cases.

Fix suggestions for redundant_pattern_matching

Fixes the problem displayed in https://github.com/rust-lang/rust-clippy/issues/4344#issuecomment-519206388.

We now append `{}` to the suggestion so that the conditional has the
correct syntax again.

(If we were to _remove_ the `if` instead, it would trigger the
`unused_must_use` warning for `#[must_use]` types.

Add note on how to find the latest beta commit

Auto merge of #4423 - lzutao:await-is-stable, r=Manishearth

Remove feature gate for async_await

async/await is stablized in https://github.com/rust-lang/rust/pull/63209 :tada:
changelog: none

Remove feature gate for async_await

Auto merge of #4419 - lzutao:ifchain-link, r=phansch

Update if_chain doc link

changelog: none

Auto merge of #4401 - JJJollyjim:literal-separation-suggestion, r=flip1995

Add autofixable suggestion for unseparated integer literal suffixes

changelog: Add autofixable suggestion for unseparated integer literal suffixes

Somewhat WIP, since I haven't been able to get this working when adding `// run-rustfix` to `ui/literals.rs`. I think the issue is that there are multiple suggestions operating on one numerical literal, and I'm not sure what the best approach is to work around that.

Thanks

Auto merge of #4420 - phansch:disable_rls_integration_test, r=phansch

Disable RLS integration test

until RLS has been updated to the latest Clippy commit.

cc https://github.com/rust-lang/rust-clippy/pull/4416#issuecomment-522859091

changelog: none

Merge pull request #4416 from SimonSapin/plugin

Import rustc_plugin from its new location

Disable RLS integration test

until RLS has been updated to the latest Clippy commit.

Update if_chain doc link

Auto merge of #4417 - kraai:remove-in_macro_or_desugar, r=phansch

Remove in_macro_or_desugar

`in_macro_or_desugar` is just a wrapper around `Span::from_expansion`, so remove the former and call the latter instead.

changelog: Remove `in_macro_or_desugar`.

Restore en dashes

Remove in_macro_or_desugar

Import rustc_plugin from its new location

Depends on https://github.com/rust-lang/rust/pull/62727

Auto merge of #4414 - chansuke:fix-contributing-doc, r=phansch

Fix the name of a channel of discord in CONTRIBUTING.md

Fixes #4400.

changelog: none

Update the naming of the Discord channel.

Fix the name of a channel of discord in CONTRIBUTING.md

Requested test cleanup

Auto merge of #4411 - mikerite:fix-4384, r=flip1995

Fix `clone_on_copy` false positives

Closes #4384

changelog: Fix `clone_on_copy` false positives

Auto merge of #4386 - lzutao:lint-option-and_then-some, r=flip1995

Add option_and_then_some lint

changelog: Add complexity lint to warn about `option.and_then(|o| Some(x))` and suggest replacing with `option.map(|o| x)`.

Closes #4299

Auto merge of #4410 - matthiaskrgr:url_serde_feat, r=phansch

deps: try to fix build in rustc repo: enable serde feature url dep in clippy-lints crate

This might fix the problem in https://github.com/rust-lang/rust/pull/63587
I didn't have time to test this yet.
r? @flip1995
changelog: enable serde feature of url dep in clippy-lints to depened on it in the same way cargo does

Fix `clone_on_copy` false positives

Closes #4384

Add test

try to fix build in rustc repo

Allow option_and_then_some in option_map_or_none test

Add option_and_then_some lint

Requested changes

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

Ignore lines starting with '#'

Run update_lints

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

Implement "suspicious_map" lint

Resolves #4010

changelog: New lint `suspicious_map`.

run-rustfix for unseparated-prefix-literals

Change lint type to 'complexity'

Add autofixable suggestion for unseparated integer literal suffices

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'