From f42be913e407ae20cd33167b881953a054420872 Mon Sep 17 00:00:00 2001 From: "Davide Gessa (dakk)" Date: Mon, 8 Jul 2024 11:12:29 +0200 Subject: [PATCH] var renaming --- qlasskit/ast2ast/astrewriter.py | 34 ++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/qlasskit/ast2ast/astrewriter.py b/qlasskit/ast2ast/astrewriter.py index 8c92eaf..c3093ea 100644 --- a/qlasskit/ast2ast/astrewriter.py +++ b/qlasskit/ast2ast/astrewriter.py @@ -242,12 +242,12 @@ def visit_If(self, node): if len(b.targets) != 1 or not isinstance(b.targets[0], ast.Name): raise Exception("if targets only allow one Name target: ", ast.dump(b)) - target_0id = b.targets[0].id + target = b.targets[0].id - if target_0id.startswith("__") and target_0id not in self.env: - orelse_inner = ast.Name(id=target_0id[2:]) + if target.startswith("__") and target not in self.env: + orelse_inner = ast.Name(id=target[2:]) else: - orelse_inner = ast.Name(id=target_0id) + orelse_inner = ast.Name(id=target) if_l.append( ast.Assign( @@ -265,15 +265,15 @@ def visit_If(self, node): if len(b.targets) != 1 or not isinstance(b.targets[0], ast.Name): raise Exception("if targets only allow one Name target: ", ast.dump(b)) - target_0id = b.targets[0].id + target = b.targets[0].id - if target_0id.startswith("__") and target_0id not in self.env: - orelse_inner = ast.Name(id=target_0id[2:]) - elif target_0id[0 : len("_iftarg")] == "_iftarg": + if target.startswith("__") and target not in self.env: + orelse_inner = ast.Name(id=target[2:]) + elif target[0 : len("_iftarg")] == "_iftarg": if_l.append(b) continue else: - orelse_inner = ast.Name(id=target_0id) + orelse_inner = ast.Name(id=target) if_l.append( ast.Assign( @@ -304,25 +304,25 @@ def visit_FunctionDef(self, node): return super().generic_visit(node) def visit_Assign(self, node): - target_0id = node.targets[0].id - was_known = target_0id in self.env + target = node.targets[0].id + was_known = target in self.env if isinstance(node.value, ast.Constant): - self.env.set_constant(target_0id, node.value) + self.env.set_constant(target, node.value) elif isinstance(node.value, ast.Name) and node.value.id in self.env: - self.env.copy_type(node.value.id, target_0id) + self.env.copy_type(node.value.id, target) elif isinstance(node.value, ast.Tuple) or isinstance(node.value, ast.List): - self.env.set_constant(target_0id, self.visit(node.value)) + self.env.set_constant(target, self.visit(node.value)) else: - self.env.set_type(target_0id, "Unknown") + self.env.set_type(target, "Unknown") # If value is not self referencing, we can skip this (ie: a = b + 1) - ip = IsNamePresent(target_0id) + ip = IsNamePresent(target) ip.visit(node.value) # Reassigning an already present variable (use a temp variable) if ip.present and was_known and not isinstance(node.value, ast.Constant): - new_targ = ast.Name(id=f"__{target_0id}", ctx=ast.Load()) + new_targ = ast.Name(id=f"__{target}", ctx=ast.Load()) return [ ast.Assign(