fix a couple of python warnings (#3526)

nirvn · m-kuhn · commit eefeef573d54 · 2016-09-23T07:36:05.000+02:00
diff --git a/python/plugins/MetaSearch/util.py b/python/plugins/MetaSearch/util.py
@@ -35,6 +35,7 @@
 from gettext import gettext, ngettext
 import logging
 import os
+import codecs
 import webbrowser
 from xml.dom.minidom import parseString
 import xml.etree.ElementTree as etree
@@ -61,7 +62,8 @@ def __init__(self):
         """init"""
         self.ppath = os.path.dirname(os.path.abspath(__file__))
         self.metadata = configparser.ConfigParser()
-        self.metadata.readfp(open(os.path.join(self.ppath, 'metadata.txt')))
+        with codecs.open(os.path.join(self.ppath, 'metadata.txt'), "r", "utf8") as f:
+            self.metadata.read_file(f)
 
 
 def get_ui_class(ui_file):
diff --git a/python/pyplugin_installer/installer_data.py b/python/pyplugin_installer/installer_data.py
@@ -580,7 +580,8 @@ def metadataParser(fct):
             global errorDetails
             cp = configparser.ConfigParser()
             try:
-                cp.read_file(codecs.open(metadataFile, "r", "utf8"))
+                with codecs.open(metadataFile, "r", "utf8") as f:
+                    cp.read_file(f)
                 return cp.get('general', fct)
             except Exception as e:
                 if not errorDetails:
diff --git a/python/utils.py b/python/utils.py
@@ -254,9 +254,8 @@ def findPlugins(path):
         cp = configparser.ConfigParser()
 
         try:
-            f = codecs.open(metadataFile, "r", "utf8")
-            cp.read_file(f)
-            f.close()
+            with codecs.open(metadataFile, "r", "utf8") as f:
+                cp.read_file(f)
         except:
             cp = None
 
