Comment #2 on issue 2970 by snktagar...@gmail.com: coverage_doctest.py should report public classes (not just functions)
http://code.google.com/p/sympy/issues/detail?id=2970

There is an issue with the existing doctest coverage tool itself. The check to find "def" and correspondingly continue till we reach the end of declaration by finding a regex of the form '\)\s*\:' does not take into account that such definitions can come inside the comments as well. For ex: If I happen to write """ def as crash it (means something) :""" in the comments, it incorrectly detects "as crash it" as a possible function name.

IMO, we need to check if a given match for def/class _token_ lies _inside_ of comments! That should solve most of the current possible issues (more so when I am trying to implement for class)

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To post to this group, send email to sympy-issues@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