Skip to content

Commit d2fed1a

Browse files
committed
less index math
1 parent 5790a83 commit d2fed1a

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

crates/ruff_linter/src/rules/flake8_simplify/rules/ast_bool_op.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -443,8 +443,11 @@ pub(crate) fn duplicate_isinstance_call(checker: &mut Checker, expr: &Expr) {
443443
let call = node2.into();
444444

445445
// Generate the combined `BoolOp`.
446-
let before = values.iter().take(indices[0]).cloned();
447-
let after = values.iter().skip(indices[indices.len() - 1] + 1).cloned();
446+
let [first, .., last] = indices.as_slice() else {
447+
unreachable!("Indices should have at least two elements")
448+
};
449+
let before = values.iter().take(*first).cloned();
450+
let after = values.iter().skip(last + 1).cloned();
448451
let node = ast::ExprBoolOp {
449452
op: BoolOp::Or,
450453
values: before.chain(iter::once(call)).chain(after).collect(),

0 commit comments

Comments
 (0)