Comment #2 on issue 2970 by [email protected]: 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 [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.
