Unnamed repository; edit this file 'description' to name the repository.
| -rw-r--r-- | Cargo.toml | 1 | ||||
| -rw-r--r-- | crates/ide-assists/src/handlers/inline_const_as_literal.rs | 2 | ||||
| -rw-r--r-- | crates/ide-db/src/active_parameter.rs | 2 |
3 files changed, 2 insertions, 3 deletions
diff --git a/Cargo.toml b/Cargo.toml index b9b3d3d8f4..35b439752b 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -177,7 +177,6 @@ format_collect = "allow" large_enum_variant = "allow" needless_doctest_main = "allow" new_without_default = "allow" -nonminimal_bool = "allow" non_canonical_clone_impl = "allow" non_canonical_partial_ord_impl = "allow" non_minimal_cfg = "allow" diff --git a/crates/ide-assists/src/handlers/inline_const_as_literal.rs b/crates/ide-assists/src/handlers/inline_const_as_literal.rs index d5ede9e3c0..1843745376 100644 --- a/crates/ide-assists/src/handlers/inline_const_as_literal.rs +++ b/crates/ide-assists/src/handlers/inline_const_as_literal.rs @@ -100,7 +100,7 @@ fn validate_type_recursively( } (_, Some(ty)) => match ty.as_builtin() { // `const A: str` is not correct, but `const A: &builtin` is. - Some(builtin) if refed || (!refed && !builtin.is_str()) => Some(()), + Some(builtin) if refed || !builtin.is_str() => Some(()), _ => None, }, _ => None, diff --git a/crates/ide-db/src/active_parameter.rs b/crates/ide-db/src/active_parameter.rs index 8d34f01b00..5780b5a5bb 100644 --- a/crates/ide-db/src/active_parameter.rs +++ b/crates/ide-db/src/active_parameter.rs @@ -23,7 +23,7 @@ impl ActiveParameter { let idx = active_parameter?; let mut params = signature.params(sema.db); - if !(idx < params.len()) { + if idx >= params.len() { cov_mark::hit!(too_many_arguments); return None; } |