Comment #35 on issue 1923 by nicolas.pourcelot: count_ops doesn't return a count (by default)
http://code.google.com/p/sympy/issues/detail?id=1923

Well, I personally don't mind.

Concerning style, however, I would write it as following :

    elif isinstance(expr, Expr):
        return expr.count_ops(visual)
    return NotImplemented

or

    elif isinstance(expr, Expr):
        return expr.count_ops(visual)
    raise NotImplementedError

Note that since Expr is a subclass of Basic, "not isinstance(expr, Expr) or not isinstance(expr, Basic)" is a verbose equivalent of "not isinstance(expr, Expr)".

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