cfgshell: Remove pylint disable line
authorMichael Hanselmann <hansmi@google.com>
Thu, 27 Sep 2012 12:52:38 +0000 (14:52 +0200)
committerMichael Hanselmann <hansmi@google.com>
Thu, 27 Sep 2012 13:02:48 +0000 (15:02 +0200)
Commit 326830963 removed the use of a private member function. Also
replace '' with "" in one place.

Signed-off-by: Michael Hanselmann <hansmi@google.com>
Reviewed-by: Iustin Pop <iustin@google.com>

tools/cfgshell

index fa145c2..90b3dcf 100755 (executable)
@@ -93,8 +93,6 @@ class ConfigShell(cmd.Cmd):
     dirs = []
     entries = []
     if isinstance(obj, objects.ConfigObject):
-      # pylint: disable=W0212
-      # yes, we're using a protected member
       for name in obj.GetAllSlots():
         child = getattr(obj, name, None)
         if isinstance(child, (list, dict, tuple, objects.ConfigObject)):
@@ -119,7 +117,7 @@ class ConfigShell(cmd.Cmd):
     configuration is loaded.
 
     """
-    if line.startswith("load") or line == 'EOF' or line == "quit":
+    if line.startswith("load") or line == "EOF" or line == "quit":
       return line
     if not self.parents or self.cfg is None:
       print "No config data loaded"