Minor refactoring

stamparm · stamparm · commit ffc520b35f21 · 2012-07-24T14:35:56.000+02:00
diff --git a/plugins/generic/filesystem.py b/plugins/generic/filesystem.py
@@ -125,36 +125,31 @@ def fileEncode(self, fileName, encoding, single):
         back-end DBMS underlying file system
         """
 
-        fcEncodedList = []
-        fp = codecs.open(fileName, "rb")
-        fcEncodedStr = fp.read().encode(encoding).replace("\n", "")
+        retVal = []
+        with codecs.open(fileName, "rb") as f:
+            content = f.read().encode(encoding).replace("\n", "")
 
         if not single:
-            fcLength = len(fcEncodedStr)
-
-            if fcLength > 256:
-                for i in xrange(0, fcLength, 256):
-                    string = ""
+            if len(content) > 256:
+                for i in xrange(0, len(content), 256):
+                    _ = content[i:i+256]
 
                     if encoding == "hex":
-                        string += "0x"
-
-                    string += fcEncodedStr[i:i+256]
-
-                    if encoding == "base64":
-                        string = "'%s'" % string
+                        _ = "0x%s" % _
+                    elif encoding == "base64":
+                        _ = "'%s'" % _
 
-                    fcEncodedList.append(string)
+                    retVal.append(_)
 
-        if not fcEncodedList:
+        if not retVal:
             if encoding == "hex":
-                fcEncodedStr = "0x%s" % fcEncodedStr
+                content = "0x%s" % content
             elif encoding == "base64":
-                fcEncodedStr = "'%s'" % fcEncodedStr
+                content = "'%s'" % content
 
-            fcEncodedList = [ fcEncodedStr ]
+            retVal = [ content ]
 
-        return fcEncodedList
+        return retVal
 
     def askCheckWrittenFile(self, wFile, dFile, fileType):
         message = "do you want confirmation that the file '%s' " % dFile
