Remove unused variables

5 files changed, 8 insertions, 30 deletions

diff --git a/miasm2/core/asmbloc.py b/miasm2/core/asmbloc.py
index a6584602..6e568740 100644
--- a/miasm2/core/asmbloc.py
+++ b/miasm2/core/asmbloc.py
@@ -1319,8 +1319,8 @@ def assemble_block(mnemo, block, symbol_pool, conservative=False):
             instr.fixDstOffset()
 
         old_l = instr.l
-        cached_candidate, candidates = conservative_asm(
-            mnemo, instr, symbol_pool, conservative)
+        cached_candidate, _ = conservative_asm(mnemo, instr, symbol_pool,
+                                               conservative)
 
         # Restore original arguments
         instr.args = saved_args
diff --git a/miasm2/core/cpu.py b/miasm2/core/cpu.py
index 050648f5..5b9462b4 100644
--- a/miasm2/core/cpu.py
+++ b/miasm2/core/cpu.py
@@ -41,7 +41,6 @@ class bitobj:
     def getbits(self, n):
         if not n:
             return 0
-        o = 0
         if n > len(self.bits) - self.offset:
             raise ValueError('not enought bits %r %r' % (n, len(self.bits)))
         b = self.bits[self.offset:self.offset + n]
@@ -174,7 +173,6 @@ def ast_parse_op(t):
         if t[0] in ['-', '+', '!']:
             return m2_expr.ExprOp(t[0], t[1])
     if len(t) == 3:
-        args = [t[0], t[2]]
         if t[1] == '-':
             # a - b => a + (-b)
             t[1] = '+'
@@ -669,9 +667,7 @@ class reg_noarg(object):
 
 
 class mn_prefix:
-
-    def __init__(self):
-        b = None
+    pass
 
 
 def swap16(v):
@@ -899,7 +895,6 @@ class metamn(type):
             bases[0].all_mn_inst[c].append(i)
             add_candidate(bases, c)
             # gen byte lookup
-            off = 0
             o = ""
             for f in i.fields_order:
                 if not isinstance(f, bsi):
@@ -996,14 +991,8 @@ class cls_mn(object):
         fname_values = pre_dis_info
         todo = [(dict(fname_values), branch, offset * 8)
                 for branch in cls.bintree.items()]
-        cpt = 0
-        if hasattr(bs, 'getlen'):
-            bs_l = bs.getlen()
-        else:
-            bs_l = len(bs)
         for fname_values, branch, offset_b in todo:
             (l, fmask, fbits, fname, flen), vals = branch
-            cpt += 1
 
             if flen is not None:
                 l = flen(attrib, fname_values)
@@ -1127,7 +1116,6 @@ class cls_mn(object):
             if not c.add_pre_dis_info(pre_dis_info):
                 continue
 
-            args = []
             todo = {}
             getok = True
             fname_values = dict(pre_dis_info)
@@ -1257,7 +1245,6 @@ class cls_mn(object):
                         raise NotImplementedError('not fully functional')
                     f.expr = expr_simp(f.expr)
                     args_expr.append(f.expr)
-                    a = args_str[start:stop]
                     args_str = args_str[stop:].strip(' ')
                     if args_str.startswith(','):
                         args_str = args_str[1:]
@@ -1341,7 +1328,7 @@ class cls_mn(object):
     @classmethod
     def filter_asm_candidates(cls, instr, candidates):
         o = []
-        for c, v in candidates:
+        for _, v in candidates:
             o += v
         o.sort(key=len)
         return o
@@ -1351,18 +1338,16 @@ class cls_mn(object):
 
         result = []
         done = []
-        cpt = 0
 
         while todo:
             index, cur_len, to_decode = todo.pop()
             # TEST XXX
-            for i, f in to_decode:
+            for _, f in to_decode:
                 setattr(self, f.fname, f)
             if (index, [x[1].value for x in to_decode]) in done:
                 continue
             done.append((index, [x[1].value for x in to_decode]))
 
-            cpt += 1
             can_encode = True
             for i, f in to_decode[index:]:
                 f.parent.l = cur_len
@@ -1377,9 +1362,7 @@ class cls_mn(object):
                 if ret is True:
                     continue
 
-                gcpt = 0
-                for i in ret:
-                    gcpt += 1
+                for _ in ret:
                     o = []
                     if ((index, cur_len, [xx[1].value for xx in to_decode]) in todo or
                         (index, cur_len, [xx[1].value for xx in to_decode]) in done):
@@ -1400,14 +1383,13 @@ class cls_mn(object):
 
     def encodefields(self, decoded):
         bits = bitobj()
-        for p, f in decoded:
+        for _, f in decoded:
             setattr(self, f.fname, f)
 
             if f.value is None:
                 continue
             bits.putbits(f.value, f.l)
 
-        xx = bits.tostring()
         return bits.tostring()
 
     def decoded2bytes(self, result):
diff --git a/miasm2/core/graph.py b/miasm2/core/graph.py
index f544757d..c98bf8a9 100644
--- a/miasm2/core/graph.py
+++ b/miasm2/core/graph.py
@@ -195,8 +195,6 @@ class DiGraph(object):
         """Render dot graph with HTML"""
 
         escape_chars = re.compile('[' + re.escape('{}') + '&|<>' + ']')
-        label_attr = 'colspan="2" align="center" bgcolor="grey"'
-        edge_attr = 'label = "%s" color="%s" style="bold"'
         td_attr = {'align': 'left'}
         nodes_attr = {'shape': 'Mrecord',
                       'fontname': 'Courier New'}
@@ -287,7 +285,6 @@ class DiGraph(object):
             dominators[node] = set(nodes)
 
         dominators[head] = set([head])
-        modified = True
         todo = set(nodes)
 
         while todo:
diff --git a/miasm2/core/parse_asm.py b/miasm2/core/parse_asm.py
index aefa6df9..b324d2cc 100644
--- a/miasm2/core/parse_asm.py
+++ b/miasm2/core/parse_asm.py
@@ -234,7 +234,6 @@ def parse_txt(mnemo, attrib, txt, symbol_pool=None):
     i = 0
     blocks = asmbloc.AsmCFG()
     block_to_nlink = None
-    block_may_link = False
     delayslot = 0
     while i < len(lines):
         if delayslot:
diff --git a/miasm2/core/types.py b/miasm2/core/types.py
index 7a4dcae1..2108d446 100644
--- a/miasm2/core/types.py
+++ b/miasm2/core/types.py
@@ -1444,7 +1444,7 @@ class MemStruct(MemType):
     @classmethod
     def _gen_attributes(cls):
         # Generate self.<name> getter and setters
-        for name, field, _ in cls._type.all_fields:
+        for name, _, _ in cls._type.all_fields:
             setattr(cls, name, property(
                 lambda self, name=name: self.get_field(name),
                 lambda self, val, name=name: self.set_field(name, val)