On Wed, Aug 25, 2010 at 3:06 PM, Eric Firing <efir...@hawaii.edu> wrote: > Looks fine to me. It's fixing a bug. I don't think the comment is even > necessary--the rationale looks pretty obvious, and the code is clear. >
Great, thanks. I'll shorten the comment to just one line then: + # Lowercase only non-module backend names (modules are case-sensitive) so that it serves as a little safety for the bug not to return, but is less verbose than before. Committed as revision 8657. Thanks! f ------------------------------------------------------------------------------ Sell apps to millions through the Intel(R) Atom(Tm) Developer Program Be part of this innovative community and reach millions of netbook users worldwide. Take advantage of special opportunities to increase revenue and speed time-to-market. Join now, and jumpstart your future. http://p.sf.net/sfu/intel-atom-d2d _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
