From 971ed9bbb320435a6353bb97efeea263d4bb2b96 Mon Sep 17 00:00:00 2001
From: Ben Boeckel <MathStuf@gmail.com>
Date: Wed, 9 Sep 2009 12:13:04 -0400
Subject: Don't return None, but {} when the section doesn't exist

---
 rpmbuild-remote.py | 31 +++++++++++++++----------------
 1 file changed, 15 insertions(+), 16 deletions(-)

(limited to 'rpmbuild-remote.py')

diff --git a/rpmbuild-remote.py b/rpmbuild-remote.py
index 82c0839..04c355e 100755
--- a/rpmbuild-remote.py
+++ b/rpmbuild-remote.py
@@ -58,23 +58,22 @@ def get_parser():
     return parser
 
 def read_section(config, section, values):
-    if not config.has_section(section):
-        return
     ret = {}
-    for value, typ in values:
-        if config.has_option(section, value):
-            if typ == 'int':
-                method = config.getint
-            elif typ == 'float':
-                method = config.getfloat
-            elif typ == 'boolean':
-                method = config.getboolean
-            elif typ == 'str':
-                method = config.get
-            else:
-                method = config.get
-                LOG.warning("Unknown type %s, assuming string" % typ)
-            ret[value] = method(section, value)
+    if config.has_section(section):
+        for value, typ in values:
+            if config.has_option(section, value):
+                if typ == 'int':
+                    method = config.getint
+                elif typ == 'float':
+                    method = config.getfloat
+                elif typ == 'boolean':
+                    method = config.getboolean
+                elif typ == 'str':
+                    method = config.get
+                else:
+                    method = config.get
+                    LOG.warning("Unknown type %s, assuming string" % typ)
+                ret[value] = method(section, value)
     return ret
 
 def read_preferences(opts):
-- 
cgit