Comment #3 on issue 1533 by ondrej.certik: printmethod must have printer  
object as first argument
http://code.google.com/p/sympy/issues/detail?id=1533

This patch changes tests, e.g.:

-        def _sympystr_(self):
-            return "foo"
-    assert spr(R(x)) == "foo"
+        def _sympystr_(self, printer):
+            return "foo(%s)" % printer._print(self.args[0])
+    assert spr(R(x)) == "foo(x)"


this is quite different. E.g. I suggest to test for both "foo" and "foo(x)"  
results.

Otherwise I think it's a good change, so +1 after this is fixed.

Ondrej

--
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