mirror of
https://github.com/astral-sh/ruff.git
synced 2026-10-28 10:45:37 +08:00
[ty] Remove single-use current-assignment conversions (#27361)
## Summary Remove single-use current-assignment conversions.
This commit is contained in:
1 parent
0e24940bd1
commit
9dfeff4c0f
1 file changed
+4
-22
@@ -3477,7 +3477,7 @@ impl<'db, 'ast> SemanticIndexBuilder<'db, 'ast> {
|
||||
*node.target,
|
||||
ast::Expr::Attribute(_) | ast::Expr::Subscript(_) | ast::Expr::Name(_)
|
||||
) {
|
||||
self.push_assignment(node.into());
|
||||
self.push_assignment(CurrentAssignment::AnnAssign(node));
|
||||
self.visit_expr(&node.target);
|
||||
self.pop_assignment();
|
||||
|
||||
@@ -3510,12 +3510,12 @@ impl<'db, 'ast> SemanticIndexBuilder<'db, 'ast> {
|
||||
}
|
||||
}
|
||||
|
||||
self.push_assignment(aug_assign.into());
|
||||
self.push_assignment(CurrentAssignment::AugAssign(aug_assign));
|
||||
self.visit_expr(target);
|
||||
self.pop_assignment();
|
||||
}
|
||||
ast::Expr::Name(_) | ast::Expr::Attribute(_) | ast::Expr::Subscript(_) => {
|
||||
self.push_assignment(aug_assign.into());
|
||||
self.push_assignment(CurrentAssignment::AugAssign(aug_assign));
|
||||
self.visit_expr(target);
|
||||
self.pop_assignment();
|
||||
}
|
||||
@@ -4671,7 +4671,7 @@ impl<'ast> Visitor<'ast> for SemanticIndexBuilder<'_, 'ast> {
|
||||
|
||||
// See https://peps.python.org/pep-0572/#differences-between-assignment-expressions-and-assignment-statements
|
||||
if node.target.is_name_expr() {
|
||||
self.push_assignment(node.into());
|
||||
self.push_assignment(CurrentAssignment::Named(node));
|
||||
self.visit_expr(&node.target);
|
||||
self.pop_assignment();
|
||||
} else {
|
||||
@@ -5189,24 +5189,6 @@ impl CurrentAssignment<'_, '_> {
|
||||
}
|
||||
}
|
||||
|
||||
impl<'ast> From<&'ast ast::StmtAnnAssign> for CurrentAssignment<'ast, '_> {
|
||||
fn from(value: &'ast ast::StmtAnnAssign) -> Self {
|
||||
Self::AnnAssign(value)
|
||||
}
|
||||
}
|
||||
|
||||
impl<'ast> From<&'ast ast::StmtAugAssign> for CurrentAssignment<'ast, '_> {
|
||||
fn from(value: &'ast ast::StmtAugAssign) -> Self {
|
||||
Self::AugAssign(value)
|
||||
}
|
||||
}
|
||||
|
||||
impl<'ast> From<&'ast ast::ExprNamed> for CurrentAssignment<'ast, '_> {
|
||||
fn from(value: &'ast ast::ExprNamed) -> Self {
|
||||
Self::Named(value)
|
||||
}
|
||||
}
|
||||
|
||||
#[derive(Default)]
|
||||
struct CurrentStatement<'ast, 'db> {
|
||||
/// A list of lambda expressions contained in this statement.
|
||||
|
||||
Reference in new issue
Block a user