Le Mar 9 juillet 2013 15:14, Nicolas Mailhot a écrit :
>
> Le Lun 8 juillet 2013 18:18, Martin Mokrejs a écrit :

>> Could you instead just test for "if not self._family"? Tests for
>> equality
>> are more expensive (that means self._family == 0 or self._family ==
>> False
>> will also trigger your return).
>
> I can test if it works. However, all the other tests in that file are
> already of the if Foo is None form, and I didn't want to change the coding
> style

And I can confirm the following patch also fixes my workload

diff -uNr matplotlib-1.2.1.orig/lib/matplotlib/font_manager.py
matplotlib-1.2.1/lib/matplotlib/font_manager.py
--- matplotlib-1.2.1.orig/lib/matplotlib/font_manager.py        2013-03-26
14:04:37.000000000 +0100
+++ matplotlib-1.2.1/lib/matplotlib/font_manager.py     2013-07-08
14:49:37.791845661 +0200
@@ -721,6 +721,8 @@
         Return the name of the font that best matches the font
         properties.
         """
+        if not self._family:
+          return rcParams['font.family'][0]
         return ft2font.FT2Font(str(findfont(self))).family_name

     def get_style(self):

Now could something similar be merged?

Regards,

-- 
Nicolas Mailhot


------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Matplotlib-users mailing list
Matplotlib-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-users

Reply via email to