Updates:
        Labels: -PassedReview NeedsReview

Comment #6 on issue 1551 by Toon.Verstraelen: StrPrinter is class sympy.printing.str and instance in sympy.core.basic
http://code.google.com/p/sympy/issues/detail?id=1551

I've tried to take into account the comments of Ronan. There were modules and tests importing str_printer from sympy.core.basic, which is very messy. I've cleaned up these situations and made slightly hidden _str_printer attributes for those classes that need it. These class attributes get filled in when sympy.printing is imported.
(see attached patch)

I will try to find a solution where monkey patching can be avoided completely.

Attachments:
        0002-Monkey-patch-modules-instead-of-classes.patch  8.8 KB

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sympy-issues?hl=en.

Reply via email to