Auto merge of #7873 - xFrednet:7869-string-index-ice, r=Manishearth

Update `str` utils to prevent ICEs and FNs

This PR reworks some string handling for lints regarding enum naming. I hope the refactoring will prevent future ICEs and help with new bug free implementations.

It might be better to review this PR by going through the commits, as `clippy_utils::camel_case` was renamed to `clippy_utils::str_utils` and then changed further. GH sadly doesn't really make the changes that obvious :upside_down_face:

Not too much more to say. Have a nice day :sun_with_face:

---

Fixes: rust-lang/rust-clippy#7869
changelog: ICE Fix: [`enum_variant_names`] #7869

Auto merge of #7874 - xFrednet:7756-move-lint, r=llogiq

Move `non_send_fields_in_send_ty` to `suspicious`

Stabilize the `non_send_fields_in_send_ty` lint and update the lint for the `allowed_scripts` configuration.

---

closes: rust-lang/rust-clippy#7756

changelog: Move `non_send_fields_in_send_ty` to `suspicious`

Move `non_send_fields_in_send_ty` to `suspicious`

Also updated one configuration for nicer formatting

Auto merge of #7855 - flip1995:changelog, r=xFrednet

Update CHANGELOG

r? `@xFrednet`  (you showed to be a reliable reviewer for this :) )

[Rendered](https://github.com/flip1995/rust-clippy/blob/changelog/CHANGELOG.md)

changelog: none

Update CHANGELOG

Auto merge of #7872 - calebcartwright:sync-use-new-branch, r=flip1995

recommend new branch or deleting branch when synching from rust

changelog: none

r? `@flip1995` - thanks again for your assistance in zulip earlier, figured I'd take a pass at incorporating your recommendation to delete or use a new branch into the excellent docs here

Refactored some string handling to prevent ICEs and FNs

docs: recommend new branch

Refactor `camel_case` util functions for new `StrIndex`

Auto merge of #7871 - camsteffen:fmt-update, r=camsteffen

Update rustfmt

changelog: none

Something changed with `rustup update nightly`

Update rustfmt

Rename `clippy_utils::camal_case` to `clippy_utils::str_utils`

Auto merge of #7867 - dswij:utils-typo, r=llogiq

Fix typo on utils/lib

Found some typo when looking through `clippy_utils/src/lib.rs` :slightly_smiling_face:

changelog: none

Fix typo on utils/lib

Auto merge of #7856 - Manishearth:impl-safety, r=xFrednet

missing_safety_doc: Handle 'implementation safety' headers as well

We hit some FPs on this in `yoke`, it's somewhat normal to mark trait impl safety with "implementation safety". We could also broaden the check for headers which contain the word "safety" somehow, or split out impl safety stuff to only apply to traits.

changelog: handle 'implementation safety' headers in `missing_safety_doc`

use a variable

Auto merge of #7857 - flip1995:fix_deploy, r=xFrednet

Fix deploy script

I broke this script in #7502, so that the stable symlink isn't generated
anymore. This reverts this change.

changelog: none

Auto merge of #7801 - aDotInTheVoid:empty-format, r=camsteffen

Make useless_format recognize format!("")

Closes #7796

changelog: [`useless_format`] Fix for false negitive for `format!("")`

Auto merge of #7850 - ThibsG:PossibleCastTruncation5395, r=camsteffen

Fix FP: no lint when cast is coming from `signum` method call for `cast_possible_truncation` lint

Fixes a FP when cast is coming from `signum` method call

fixes: #5395

changelog: [`cast_possible_truncation`] Fix FP when cast is coming from `signum` method call

Make useless_format recognize format!("")

Closes #7796

Do not lint when cast is coming from `signum` method call

Fix deploy script

I broke this script in #7502, so that the stable symlink isn't generated
anymore. This reverts this change.

Handle 'implementation safety' headers as well

Auto merge of #7853 - flip1995:rustup, r=flip1995

Rustup

r? `@ghost`

changelog: none

Bump nightly version -> 2021-10-21

Bump Clippy Version -> 0.1.58

Merge remote-tracking branch 'upstream/master' into rustup

Auto merge of #7852 - surechen:fix_typo, r=xFrednet

Fix typo for invalid_null_ptr_usage and missing_inline_in_public_items.

Fix typo for invalid_null_ptr_usage and missing_inline_in_public_items.

changelog: none

Fix typo for INVALID_NULL_PTR_USAGE and MISSING_INLINE_IN_PUBLIC_ITEMS.

Auto merge of #7838 - nhamovitz:trailing_zs_arr_wo_repr, r=Manishearth

Warn on structs with a trailing zero-sized array but no `repr` attribute

Closes #2868

changelog: Implement ``[`trailing_empty_array`]``, which warns if a struct is defined where the last field is a zero-sized array but there are no `repr` attributes. Zero-sized arrays aren't very useful in Rust itself, so such a struct is likely being created to pass to C code or in some other situation where control over memory layout matters. Either way, a `repr` attribute is needed.

Auto merge of #7849 - ThibsG:SafetyDoc, r=llogiq

FIx FP in `missing_safety_doc` lint

Fix FP where lint souldn't fire if any parent has `#[doc(hidden)]` attribute

fixes: #7347

changelog: [`missing_safety_doc`] Fix FP if any parent has `#[doc(hidden)]` attribute

Do not lint if any parent has hidden attribute

Auto merge of #7848 - Veykril:patch-1, r=xFrednet

Fix doc heading of `transmute_num_to_bytes`

changelog: none

Fix doc heading of TRANSMUTE_NUM_TO_BYTES

Auto merge of #7775 - F3real:no_effect, r=xFrednet

Detect underscored variables with no side effects

Fixes #7545

changelog: Lint on underscored variables with no side effects in [`no_effect`]

Auto merge of #7840 - dswij:question-mark-result, r=llogiq

Cover `Result` for `question_mark`

closes #7135

changelog: [`question_mark`] now covers `Result`

Small refactor on `question_mark` condition checks

Auto merge of #7833 - Dmitry-Borodin:patch-1, r=giraffate

Add reference to another doc with explanation

Add reference to another doc that explains which repository should be passed in this command since this is not covered in the command help itself.
changelog: none

Lint on underscore variable assignment

Fix tests after no_effect update

Add a drop testcase

Don't lint _ variables in macro expansion

Address review comments and update tests

Don't shadow unnecessary operation lint if no_effect is allowed

Revert shadowing change and remove no_effect allows

Update clippy_lints/src/no_effect.rs

Co-authored-by: Takayuki Nakata <f.seasons017@gmail.com>
Update clippy_lints/src/no_effect.rs

Co-authored-by: Takayuki Nakata <f.seasons017@gmail.com>
Address review comments

Rename lint

remove resolved note

Auto merge of #7811 - rust-lang:eq-op-testless, r=xFrednet

avoid `eq_op` in test code

Add a check to `eq_op` that will avoid linting in functions annotated with `#[test]`

---

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: avoid `eq_op` in test functions

Rollup merge of #88860 - nbdd0121:panic, r=m-ou-se

Deduplicate panic_fmt

std's begin_panic_fmt and core's panic_fmt are duplicates. Merge them to declutter code and remove a lang item.

avoid `eq_op` in test code

made description of intellij a link by moving outside of code block.

Remove begin_panic_fmt from clippy

Update test for `question_mark` to cover `Result`

Cover `Result` for `question_mark`

formatting 🙃

Use real type in doc examples

Remove explicit lifetime

Remove comment

One more test + final tidying

Revert "!: this is the commit that demonstrates the ICE"

This reverts commit d85f903c91d909534003ee2ff0e16316b20687dc.

!: this is the commit that demonstrates the ICE

Don't need `rustc_attr` anymore

Improve help message

All five `has_repr_attr` agree + are correct

Exploring emitting other sorts of `span`s

Better testcase names

One more test

Still renaming lmao

Improve doc and span messages

Rename stderr

Rename lint

run rustfmt

Tidy import + update expected stderr

Tidy imports

Check for tuple structs

Refactor ZS array detection again and this one seems great 👍

update testsuite and expand `if_chain`

Tidy comments + tests; revert 'size-is-zero' detection

tried to simplify but it doesn't work :/

intermediate step

add multiple `get_attrs` and `includes_repr` and they all work!

Ran `dev bless`!

output looks fantastic

it works i think (incl some `dbg`s)

Implement getting an array of attributes!

Implement detecting trailing zero-sized array

ayy it compiles! ship it, right? 😎 /s

why was `rustc_lint_defs` not already externed in `lib.rs`?
and how was r-a able to find it but cargo wasn't? 🤔

Well it builds

Add boilerplate and basic tests

Auto merge of #7835 - mikerite:unneeded-allow-20211018, r=xFrednet

Remove unneeded allow

Remove unneeded allow

changelog: none

Remove unneeded allow

Rollup merge of #89990 - petrochenkov:idempty, r=wesleywiser

rustc_span: `Ident::invalid` -> `Ident::empty`

The equivalent for `Symbol`s was renamed some time ago (`kw::Invalid` -> `kw::Empty`), and it makes sense to do the same thing for `Ident`s as well.

Auto merge of #7832 - narpfel:implicit-saturating-sub-false-positive-else, r=giraffate

Fix false positive of `implicit_saturating_sub` with `else` clause

Fixes #7831

changelog: Fix false positive of [`implicit_saturating_sub`] with `else` clause

rustc_span: `Ident::invalid` -> `Ident::empty`

The equivalent for `Symbol`s was renamed some time ago (`kw::Invalid` -> `kw::Empty`), and it makes sense to do the same thing for `Ident`s.

Add reference to another doc with explanation

Add reference to another doc that explains which repository should be passed in this command since this is not covered in the command help itself.

Rollup merge of #89963 - r00ster91:parenthesisparentheses, r=nagisa

Some "parenthesis" and "parentheses" fixes

"Parenthesis" is the singular (e.g. one `(` or one `)`) and "parentheses" is the plural (multiple `(` or `)`s) and this is not hard to mix up so here are some fixes for that.

Inspired by #89958

Fix false positive of `implicit_saturating_sub` with `else` clause

Fixes #7831

Some "parenthesis" and "parentheses" fixes

Auto merge of #7827 - zvavybir:master, r=xFrednet

Fixed naive doc formatting for `#[must_use]` lints

The documentation of a few lints around the `#[must_use]` attribute had a few times "[`#[must_use]`]" (without the parentheses, but with the `[]`) and once the "`" was missing.

changelog: Fixed naive doc formatting for `#[must_use]` lints ([`must_use_unit`], [`double_must_use`], [`must_use_candidate`], [`let_underscore_must_use`])

Fixed naive doc formatting for `#[must_use]` lints

Auto merge of #7743 - smoelius:master, r=camsteffen

Add `format_in_format_args` and `to_string_in_format_args` lints

Fixes #7667 and #7729

I put these in `perf` since that was one of `@jplatte's` suggestions, and `redundant_clone` (which I consider to be similar) lives there as well.

However, I am open to changing the category or anything else.

r? `@camsteffen`

changelog: Add `format_in_format_args` and `to_string_in_format_args` lints

Remove redundant `to_string`s

Fix adjacent tests

Add `format_in_format_args` and `to_string_in_format_args` lints

Fixes #7667 and #7729

Auto merge of #7823 - mikerite:match-overlapping-arm-20211015, r=xFrednet

Some small improvements to the `match_overlapping_arm` tests

changelog: none

Auto merge of #7824 - dswij:unnecessary_sort_by, r=llogiq

`unnecessary_sort_by` checks if argument implements `Ord` trait

closes #7822

changelog: [`unnecessary_sort_by`] now checks if argument implements `Ord` trait