Try to fix .fixed

Fix tests

Use the spans returned by utils::method_calls

Also return the method spans in utils::method_calls

Dereference one less on search_is_some and make it auto-fixable

Auto merge of #4408 - phansch:more_rustfix_tests, r=flip1995

More rustfix tests

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

cc #3630

This is probably easier reviewed per-commit.

changelog: none

Auto merge of #4469 - vityafx:master, r=flip1995

Improve the cognitive_complexity lint's warning.

Adds the value of complexity limit set for the lint
to the warning.

Fixes #4466

Signed-off-by: Victor Polevoy <fx@thefx.co>
changelog: improve the `cognitive_complexity` lint's warning message.

Improve cognitive_complexity lint's warning.

Adds the value of complexity limit set for the lint
to the warning.

Fixes #4466

Signed-off-by: Victor Polevoy <fx@thefx.co>

Make useless_attribute suggestion MaybeIncorrect

Auto merge of #4465 - JohnTitor:add-note-to-fix-toolstate, r=flip1995

Add note to fix toolstate

The idea from https://github.com/rust-lang/rust-clippy/pull/4462#issuecomment-525725587
It's nice that we have the documentation to guide fix toolstate.

changelog: none

Add note to fix toolstate

Auto merge of #4464 - flip1995:fetch_prs_between, r=phansch

Also fetch rollup merges in fetch_prs_between.sh script

Otherwise rolled up PRs won't be included in the changelog, e.g. 236666138fa15fdd48b1c12180b2c9dc936441b4

changelog: none

Also fetch rollup merges in fetch_prs_between.sh script

Auto merge of #4463 - flip1995:rollup-240kr2c, r=flip1995

Rollup of 2 pull requests

Successful merges:

 - #4459 (Add note to avoid confusing)
 - #4460 (Fix `inherent_to_string` false positive)

Failed merges:

r? @ghost

changelog: none

Rollup merge of #4460 - JohnTitor:fix-inherent-to-string, r=flip1995

Fix `inherent_to_string` false positive

Fixes #4457

changelog: fixes `inherent_to_string` false positive

Rollup merge of #4459 - JohnTitor:update-testing-notes, r=flip1995

Add note to avoid confusing

Fixes #4456

changelog: none

Auto merge of #4462 - JohnTitor:fix-build-arg, r=flip1995

Replace `Arg` with `Param`

Fix build issue.
Rustup to https://github.com/rust-lang/rust/pull/63127

changelog: none

Replace `Arg` with `Param`

Update int_plus_one suggestion text

This is now in line with the recommendations from here:
https://doc.rust-lang.org/nightly/nightly-rustc/rustc_errors/struct.Diagnostic.html#method.span_suggestion

Change float_cmp Applicability to `HasPlaceholders`

The suggestion includes an 'errors' placeholder currently, so it's not
`MachineApplicable`

Add run-rustfix for rename test

I removed the `Bar` static and `Foo` impl because they are not needed
for what's tested here as far as I can tell.

Add run-rustfix for outer_expn_data

Add run-rustfix for useless_attribute test

Add run-rustfix for int_plus_one test

Add run-rustfix for assign_ops test

Fix `inherent_to_string` false positive

Add note to avoid confusing

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`