From f267073e0d03140fa3fc48aabc1d94465fbc2ec8 Mon Sep 17 00:00:00 2001 From: Daniel Dehennin Date: Tue, 29 Jan 2013 15:21:31 +0100 Subject: [PATCH] Variable "ret" referenced before assignment 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 | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tiramisu/option.py b/tiramisu/option.py index 7c16771..a21a0fd 100644 --- a/tiramisu/option.py +++ b/tiramisu/option.py @@ -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: