From 6a1ee6c86a3d0c4a343b2f22fa5f6328002ea047 Mon Sep 17 00:00:00 2001 From: Ulf Magnusson Date: Sat, 6 Jun 2015 10:22:38 +0200 Subject: load_config() nit. Each expression in the loop is relatively expensive at this point too. --- kconfiglib.py | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'kconfiglib.py') diff --git a/kconfiglib.py b/kconfiglib.py index 9ff4bbf..0899e10 100644 --- a/kconfiglib.py +++ b/kconfiglib.py @@ -305,10 +305,8 @@ class Config(object): .replace('\\"', '"').replace("\\\\", "\\") if name in self.syms: sym = self.syms[name] - - old_user_val = sym.user_val - if old_user_val is not None: - warn_override(filename, linenr, name, old_user_val, val) + if sym.user_val is not None: + warn_override(filename, linenr, name, sym.user_val, val) if sym.is_choice_symbol_: user_mode = sym.parent.user_mode @@ -318,7 +316,6 @@ class Config(object): .format(name, val, user_mode), filename, linenr) - sym._set_user_value_no_invalidate(val, True) else: @@ -334,10 +331,8 @@ class Config(object): name = unset_match.group(1) if name in self.syms: sym = self.syms[name] - - old_user_val = sym.user_val - if old_user_val is not None: - warn_override(filename, linenr, name, old_user_val, "n") + if sym.user_val is not None: + warn_override(filename, linenr, name, sym.user_val, "n") sym._set_user_value_no_invalidate("n", True) -- cgit v1.2.3