Memoize parse_fontconfig_pattern; speeds up test suite by ~1min.

anntzer · anntzer · commit 263ac2ad2e80 · 2017-03-09T16:24:33.000-08:00
On my laptop this patch drops the duration of the test suite from 604s
to 554s (with the inkscape patch on as well).
diff --git a/lib/matplotlib/fontconfig_pattern.py b/lib/matplotlib/fontconfig_pattern.py
@@ -21,9 +21,15 @@
 
 import six
 
-import re, sys
-from pyparsing import Literal, ZeroOrMore, \
-     Optional, Regex, StringEnd, ParseException, Suppress
+import re
+import sys
+from pyparsing import (Literal, ZeroOrMore, Optional, Regex, StringEnd,
+                       ParseException, Suppress)
+
+try:
+    from functools import lru_cache
+except ImportError:
+    from backports.functools_lru_cache import lru_cache
 
 family_punc = r'\\\-:,'
 family_unescape = re.compile(r'\\([%s])' % family_punc).sub
@@ -166,7 +172,11 @@ def _property(self, s, loc, tokens):
             self._properties.setdefault(key, []).extend(val)
         return []
 
-parse_fontconfig_pattern = FontconfigPatternParser().parse
+# `parse_fontconfig_pattern` is a bottleneck during the tests because it is
+# repeatedly called when the rcParams are reset (to validate the default
+# fonts).  In practice, the cache size doesn't grow beyond a few dozen entries
+# during the test suite.
+parse_fontconfig_pattern = lru_cache()(FontconfigPatternParser().parse)
 
 def generate_fontconfig_pattern(d):
     """
