Use always_iterable to re-use a proven function for the pattern of accepting None, iterable of strings, or single string for a parameter.

1 files changed, 3 insertions, 3 deletions

diff --git a/cherrypy/lib/reprconf.py b/cherrypy/lib/reprconf.py
index 8af1f777..7e8cb936 100644
--- a/cherrypy/lib/reprconf.py
+++ b/cherrypy/lib/reprconf.py
@@ -44,6 +44,8 @@ except ImportError:
 import operator as _operator
 import sys
 
+from jaraco.itertools import always_iterable
+
 
 def as_dict(config):
     """Return a dict from 'config' whether it is a dict, file, or filename."""
@@ -192,9 +194,7 @@ class Parser(ConfigParser):
         return optionstr
 
     def read(self, filenames):
-        if isinstance(filenames, basestring):
-            filenames = [filenames]
-        for filename in filenames:
+        for filename in always_iterable(filenames):
             # try:
             #     fp = open(filename)
             # except IOError: