mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-11-28 07:30:57 +00:00
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`
This commit is contained in:
commit
4f46f4266a
6 changed files with 7 additions and 5 deletions
|
@ -218,6 +218,7 @@ store.register_group(true, "clippy::all", Some("clippy_all"), vec![
|
||||||
LintId::of(non_copy_const::DECLARE_INTERIOR_MUTABLE_CONST),
|
LintId::of(non_copy_const::DECLARE_INTERIOR_MUTABLE_CONST),
|
||||||
LintId::of(non_expressive_names::JUST_UNDERSCORES_AND_DIGITS),
|
LintId::of(non_expressive_names::JUST_UNDERSCORES_AND_DIGITS),
|
||||||
LintId::of(non_octal_unix_permissions::NON_OCTAL_UNIX_PERMISSIONS),
|
LintId::of(non_octal_unix_permissions::NON_OCTAL_UNIX_PERMISSIONS),
|
||||||
|
LintId::of(non_send_fields_in_send_ty::NON_SEND_FIELDS_IN_SEND_TY),
|
||||||
LintId::of(open_options::NONSENSICAL_OPEN_OPTIONS),
|
LintId::of(open_options::NONSENSICAL_OPEN_OPTIONS),
|
||||||
LintId::of(option_env_unwrap::OPTION_ENV_UNWRAP),
|
LintId::of(option_env_unwrap::OPTION_ENV_UNWRAP),
|
||||||
LintId::of(overflow_check_conditional::OVERFLOW_CHECK_CONDITIONAL),
|
LintId::of(overflow_check_conditional::OVERFLOW_CHECK_CONDITIONAL),
|
||||||
|
|
|
@ -17,7 +17,6 @@ store.register_group(true, "clippy::nursery", Some("clippy_nursery"), vec![
|
||||||
LintId::of(missing_const_for_fn::MISSING_CONST_FOR_FN),
|
LintId::of(missing_const_for_fn::MISSING_CONST_FOR_FN),
|
||||||
LintId::of(mutable_debug_assertion::DEBUG_ASSERT_WITH_MUT_CALL),
|
LintId::of(mutable_debug_assertion::DEBUG_ASSERT_WITH_MUT_CALL),
|
||||||
LintId::of(mutex_atomic::MUTEX_INTEGER),
|
LintId::of(mutex_atomic::MUTEX_INTEGER),
|
||||||
LintId::of(non_send_fields_in_send_ty::NON_SEND_FIELDS_IN_SEND_TY),
|
|
||||||
LintId::of(nonstandard_macro_braces::NONSTANDARD_MACRO_BRACES),
|
LintId::of(nonstandard_macro_braces::NONSTANDARD_MACRO_BRACES),
|
||||||
LintId::of(option_if_let_else::OPTION_IF_LET_ELSE),
|
LintId::of(option_if_let_else::OPTION_IF_LET_ELSE),
|
||||||
LintId::of(path_buf_push_overwrite::PATH_BUF_PUSH_OVERWRITE),
|
LintId::of(path_buf_push_overwrite::PATH_BUF_PUSH_OVERWRITE),
|
||||||
|
|
|
@ -15,6 +15,7 @@ store.register_group(true, "clippy::suspicious", Some("clippy_suspicious"), vec!
|
||||||
LintId::of(loops::MUT_RANGE_BOUND),
|
LintId::of(loops::MUT_RANGE_BOUND),
|
||||||
LintId::of(methods::SUSPICIOUS_MAP),
|
LintId::of(methods::SUSPICIOUS_MAP),
|
||||||
LintId::of(mut_key::MUTABLE_KEY_TYPE),
|
LintId::of(mut_key::MUTABLE_KEY_TYPE),
|
||||||
|
LintId::of(non_send_fields_in_send_ty::NON_SEND_FIELDS_IN_SEND_TY),
|
||||||
LintId::of(suspicious_trait_impl::SUSPICIOUS_ARITHMETIC_IMPL),
|
LintId::of(suspicious_trait_impl::SUSPICIOUS_ARITHMETIC_IMPL),
|
||||||
LintId::of(suspicious_trait_impl::SUSPICIOUS_OP_ASSIGN_IMPL),
|
LintId::of(suspicious_trait_impl::SUSPICIOUS_OP_ASSIGN_IMPL),
|
||||||
])
|
])
|
||||||
|
|
|
@ -44,7 +44,7 @@ declare_clippy_lint! {
|
||||||
/// Use thread-safe types like [`std::sync::Arc`](https://doc.rust-lang.org/std/sync/struct.Arc.html)
|
/// Use thread-safe types like [`std::sync::Arc`](https://doc.rust-lang.org/std/sync/struct.Arc.html)
|
||||||
/// or specify correct bounds on generic type parameters (`T: Send`).
|
/// or specify correct bounds on generic type parameters (`T: Send`).
|
||||||
pub NON_SEND_FIELDS_IN_SEND_TY,
|
pub NON_SEND_FIELDS_IN_SEND_TY,
|
||||||
nursery,
|
suspicious,
|
||||||
"there is field that does not implement `Send` in a `Send` struct"
|
"there is field that does not implement `Send` in a `Send` struct"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -288,10 +288,10 @@ define_Conf! {
|
||||||
///
|
///
|
||||||
/// The list of imports to always rename, a fully qualified path followed by the rename.
|
/// The list of imports to always rename, a fully qualified path followed by the rename.
|
||||||
(enforced_import_renames: Vec<crate::utils::conf::Rename> = Vec::new()),
|
(enforced_import_renames: Vec<crate::utils::conf::Rename> = Vec::new()),
|
||||||
/// Lint: RESTRICTED_SCRIPTS.
|
/// Lint: DISALLOWED_SCRIPT_IDENTS.
|
||||||
///
|
///
|
||||||
/// The list of unicode scripts allowed to be used in the scope.
|
/// The list of unicode scripts allowed to be used in the scope.
|
||||||
(allowed_scripts: Vec<String> = vec!["Latin".to_string()]),
|
(allowed_scripts: Vec<String> = ["Latin"].iter().map(ToString::to_string).collect()),
|
||||||
/// Lint: NON_SEND_FIELDS_IN_SEND_TY.
|
/// Lint: NON_SEND_FIELDS_IN_SEND_TY.
|
||||||
///
|
///
|
||||||
/// Whether to apply the raw pointer heuristic to determine if a type is `Send`.
|
/// Whether to apply the raw pointer heuristic to determine if a type is `Send`.
|
||||||
|
|
|
@ -564,7 +564,8 @@ in the following steps:
|
||||||
/// <The configuration field doc comment>
|
/// <The configuration field doc comment>
|
||||||
(configuration_ident: Type = DefaultValue),
|
(configuration_ident: Type = DefaultValue),
|
||||||
```
|
```
|
||||||
The doc comment will be automatically added to the lint documentation.
|
The doc comment is automatically added to the documentation of the listed lints. The default
|
||||||
|
value will be formatted using the `Debug` implementation of the type.
|
||||||
2. Adding the configuration value to the lint impl struct:
|
2. Adding the configuration value to the lint impl struct:
|
||||||
1. This first requires the definition of a lint impl struct. Lint impl structs are usually
|
1. This first requires the definition of a lint impl struct. Lint impl structs are usually
|
||||||
generated with the `declare_lint_pass!` macro. This struct needs to be defined manually
|
generated with the `declare_lint_pass!` macro. This struct needs to be defined manually
|
||||||
|
|
Loading…
Reference in a new issue