Merge pull request #22487 from anntzer/mtpc

timhoffm · web-flow · commit 2661dc9518a5 · 2022-02-19T09:01:27.000+01:00
Don't key MathTextParser cache off a mutable FontProperties.
diff --git a/lib/matplotlib/mathtext.py b/lib/matplotlib/mathtext.py
@@ -233,9 +233,10 @@ def parse(self, s, dpi=72, prop=None):
         The results are cached, so multiple calls to `parse`
         with the same expression should be fast.
         """
-        # lru_cache can't decorate parse() directly because the ps.useafm and
-        # mathtext.fontset rcParams also affect the parse (e.g. by affecting
-        # the glyph metrics).
+        # lru_cache can't decorate parse() directly because prop
+        # is mutable; key the cache using an internal copy (see
+        # text._get_text_metrics_with_cache for a similar case).
+        prop = prop.copy() if prop is not None else None
         return self._parse_cached(s, dpi, prop)
 
     @functools.lru_cache(50)
