Variable "ret" referenced before assignment
authorDaniel Dehennin <daniel.dehennin@ac-dijon.fr>
Tue, 29 Jan 2013 14:21:31 +0000 (15:21 +0100)
committerDaniel Dehennin <daniel.dehennin@ac-dijon.fr>
Tue, 29 Jan 2013 14:21:31 +0000 (15:21 +0100)
Variable defined in sub-scope ("for", "if") are not defined above.

* tiramisu/option.py (Multi.append): Define "ret" to None.

* tiramisu/option.py (Multi._setvalue): Ditoo.

* tiramisu/option.py (Multi.pop): Ditoo.

tiramisu/option.py

index 7c16771..a21a0fd 100644 (file)
@@ -84,6 +84,7 @@ class Multi(list):
         except TypeError:
             return self._setvalue(value, who=settings.get_owner())
         multis = []
+        ret = None
         for name, multi in self.config:
             multis.append(multi)
         for multi in multis:
@@ -106,6 +107,7 @@ class Multi(list):
                 raise ConfigError("invalid value {0} "
                     "for option {1}".format(str(value), self.opt._name))
         oldvalue = list(self)
+        ret = None
         if key is None:
             ret = super(Multi, self).append(value)
         else:
@@ -131,6 +133,7 @@ class Multi(list):
             return self._pop(key)
 
         multis = []
+        ret = None
         for name, multi in self.config:
             multis.append(multi)
         for multi in multis: