cache regexp and PEP8

IPython/core/completer.py

@@ -497,6 +497,12 b' class IPCompleter(Completer):'
         else:
             self.clean_glob = self._clean_glob
 
+        #regexp to parse docstring for function signature
+        self.docstring_sig_re = re.compile(r'^[\w|\s.]+\(([^)]*)\).*')
+        self.docstring_kwd_re = re.compile(r'[\s|\[]*(\w+)(?:\s*=\s*.*)')
+        #use this if positional argument name is also needed
+        #= re.compile(r'[\s|\[]*(\w+)(?:\s*=?\s*.*)')
+
         # All active matcher routines for completion
         self.matchers = [self.python_matches,
                          self.file_matches,
@@ -670,24 +676,23 b' class IPCompleter(Completer):'
         form 'min(iterable[, key=func])\n' to find
         keyword argument names.
         """
-        if doc is None: return []
-        doc = doc.lstrip()
-        sio = StringIO.StringIO(doc)
+        if doc is None:
+            return []
+        sio = StringIO.StringIO(doc.lstrip())
         #care only the firstline
         #docstring can be long
         line = sio.readline()
-        p = re.compile(r'^[\w|\s.]+\(([^)]*)\).*')
+        #p = re.compile(r'^[\w|\s.]+\(([^)]*)\).*')
         #'min(iterable[, key=func])\n' -> 'iterable[, key=func]'
-        sig = p.search(line)
-        if sig is None: return []
+        sig = self.docstring_sig_re.search(line)
+        if sig is None:
+            return []
         # iterable[, key=func]' -> ['iterable[' ,' key=func]']
         sig = sig.groups()[0].split(',')
-        #use this if you want iterable to show up too
-        #q = re.compile('[\s|\[]*(\w+)(?:\s*=?\s*.*)')
-        q = re.compile('[\s|\[]*(\w+)(?:\s*=\s*.*)')
         ret = []
         for s in sig:
-            ret += q.findall(s)
+            #re.compile(r'[\s|\[]*(\w+)(?:\s*=\s*.*)')
+            ret += self.docstring_kwd_re.findall(s)
         return ret
 
     def _default_arguments(self, obj):