Comment #6 on issue 2114 by smichr: Mul.make_args(S.One) returns ()
http://code.google.com/p/sympy/issues/detail?id=2114

This is stalled...how can we come to a final decision on this? I'm writing code to work around this and I would rather just have it one way or another.

I still think you should get args no matter what. e.g. let's say you want to iterate through the terms of an Add and print the first argument of each:

current:
if not expr:
  print S.Zero
else:
  for a in Add.make_args(expr):
    for m in Mul.make_args(a):
      print m or S.One

proposed:
for a in Add.make_args(expr):
  for m in Mul.make_args(a):
    print m

It's just one extra little thing to bite you and clutter up the code if you have to remember that make_args is not always going to give you something.

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

Reply via email to