Resolve mathtext.fontset at FontProperties creation time.

anntzer · anntzer · commit 38003c06344c · 2020-11-01T19:42:16.000+01:00
... instead of delaying it until rendering time.  This makes it
consistent with the behavior of `rcParams["font.family"]`.  (While lazy
resolution may be useful too, it seems better to be consistent with
family, whose behavior is constrained by backcompat, whereas
mathtext_fontfamily has not been released yet.)
diff --git a/lib/matplotlib/font_manager.py b/lib/matplotlib/font_manager.py
@@ -897,8 +897,6 @@ def get_math_fontfamily(self):
 
         The default font is :rc:`mathtext.fontset`.
         """
-        if self._math_fontfamily is None:
-            return rcParams['mathtext.fontset']
         return self._math_fontfamily
 
     def set_math_fontfamily(self, fontfamily):
@@ -921,13 +919,12 @@ def set_math_fontfamily(self, fontfamily):
         .text.Text.get_math_fontfamily
         """
         if fontfamily is None:
-            self._math_fontfamily = None
-            return
-
-        valid_fonts = _validators['mathtext.fontset'].valid.values()
-        # _check_in_list() Validates the parameter math_fontfamily as
-        # if it were passed to rcParams['mathtext.fontset']
-        _api.check_in_list(valid_fonts, math_fontfamily=fontfamily)
+            fontfamily = rcParams['mathtext.fontset']
+        else:
+            valid_fonts = _validators['mathtext.fontset'].valid.values()
+            # _check_in_list() Validates the parameter math_fontfamily as
+            # if it were passed to rcParams['mathtext.fontset']
+            _api.check_in_list(valid_fonts, math_fontfamily=fontfamily)
         self._math_fontfamily = fontfamily
 
     def copy(self):
