Auto merge of #4444 - phansch:split_up_cmp_owned2, r=flip1995

Split up cmp_owned tests, add run-rustfix

Some of the cmp_owned tests emitted non-machine-applicable suggestions,
so I moved them to `tests/ui/cmp_owned/without_suggestion.rs` and added
`// run-rustfix` to the other half.

changelog: none

cc #3630

Auto merge of #4445 - phansch:fix_unused_unit_sugg, r=flip1995

Fix unused_unit false positive

changelog: Fix `unused_unit` false positive

For some reason the `expr` of `stmt.node` didn't contain the expansion information, but the `stmt.span` does.

Fixes #4076

Use correct path in comment

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

Auto merge of #4430 - lzutao:defid_trait_alias, r=flip1995

Account for trait alias when looking for defid

I hit the crash on the `expect` call when running clippy on rustc libcore.
Hopefully this will fix it.
changelog: none

Add regression test for looking for trait defid in nocore

Auto merge of #4436 - BO41:written_as, r=phansch

Add some "could be written as" examples

fixes #4405
changelog: none

Auto merge of #4421 - lzutao:unsep_literal, r=flip1995

Cleaner code for unsep literals

Continuing discussion in https://github.com/rust-lang/rust-clippy/pull/4401#issuecomment-522498754
changelog: none
r? @flip1995

Auto merge of #4451 - hvenev:non-exec, r=flip1995

Mark rust source file as non-executable.

It would be nice if you can backport this to beta as well.

changelog: none

Auto merge of #4431 - phansch:more_rustfix, r=flip1995

Add 3 more run-rustfix headers

changelog: none

cc #3630

Cleaner code for unsep literals

Remove empty *.std{err,out} files

Mark rust source file as non-executable.

Account for trait alias when looking for defid

Auto merge of #4447 - phansch:fix_build, r=matthiaskrgr

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

changelog: none

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

Fix unused_unit false positive

Fixes #4076

Fix lint doc

Split up cmp_owned tests, add run-rustfix

Some of the cmp_owned tests emitted non-machine-applicable suggestions,
so I moved them to `tests/ui/cmp_owned/without_suggestion.rs` and added
`// run-rustfix` to the other half.

cc #3630

Auto merge of #4443 - jeremystucki:methods-refactoring, r=phansch

Small refactoring of methods/mod.rs

changelog: none

Refactor some minor things

Use more if-chains

Refactor 'lint_or_fun_call'

Refactor 'check_unwrap_or_default'

Refactor 'check_impl_item'

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

Add examples to some complexity lints

Add example to a perf lint

Add examples to some correctness lints

Add run-rustfix for range_plus_minus_one tests

Add run-rustfix for or_fun_call tests

Add run-rustfix for short_circuit_statement test

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