More robust FontProperties hashing / copying

scottshambaugh · scottshambaugh · commit 70dc0d12b615 · 2026-02-03T14:59:20.000-07:00
diff --git a/lib/matplotlib/font_manager.py b/lib/matplotlib/font_manager.py
@@ -701,15 +701,7 @@ def _from_any(cls, arg):
             return cls(**arg)
 
     def __hash__(self):
-        l = (tuple(self.get_family()),
-             self.get_slant(),
-             self.get_variant(),
-             self.get_weight(),
-             self.get_stretch(),
-             self.get_size(),
-             self.get_file(),
-             self.get_math_fontfamily())
-        return hash(l)
+        return hash(tuple(self.__dict__.values()))
 
     def __eq__(self, other):
         return hash(self) == hash(other)
@@ -725,7 +717,7 @@ def get_family(self):
         from their respective rcParams when searching for a matching font) in
         the order of preference.
         """
-        return self._family
+        return list(self._family)
 
     def get_name(self):
         """
@@ -794,8 +786,8 @@ def set_family(self, family):
         """
         family = mpl._val_or_rc(family, 'font.family')
         if isinstance(family, str):
-            family = [family]
-        self._family = family
+            family = (family,)
+        self._family = tuple(family)
 
     def set_style(self, style):
         """
@@ -958,14 +950,7 @@ def set_math_fontfamily(self, fontfamily):
     def __copy__(self):
         # Bypass __init__ for speed, since values are already validated
         new = FontProperties.__new__(FontProperties)
-        new._family = self._family
-        new._slant = self._slant
-        new._variant = self._variant
-        new._weight = self._weight
-        new._stretch = self._stretch
-        new._size = self._size
-        new._file = self._file
-        new._math_fontfamily = self._math_fontfamily
+        new.__dict__.update(self.__dict__)
         return new
 
     def copy(self):
