#17404: long time doctests in multi-line code not detected
-------------------------------------------------+-------------------------
Reporter: dkrenn | Owner:
Type: enhancement | Status:
Priority: major | needs_work
Component: doctest framework | Milestone: sage-6.5
Keywords: doctest, multiline code, long | Resolution:
time | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
-------------------------------------------------+-------------------------
Changes (by dkrenn):
* status: needs_review => needs_work
* type: defect => enhancement
* milestone: sage-duplicate/invalid/wontfix => sage-6.5
Comment:
Replying to [comment:1 jdemeyer]:
> Feature, not a bug.
Ok. It's now an enhancement...
> Requiring doctest tags to be on the first line makes them much easier to
see.
I'm not happy with this. Comments within one code statement can be
confusing.
IMHO both should work, i.e., putting "# long time" in the first line as
well as in the last line (or even in any line at the end).
This would also support the following: When writing doctests I often start
with a one-liner. Then later, when it works, I'll rewrite it to multiple
lines. The current behavior means, I have to change where the comment is.
The reverse situation also happens: You have a multi-line doctest, but
then you change it and suddenly it fits into one line. Thus, you now have
to move the "# long time" from somewhere in betweeb to the end of the
line.
--
Ticket URL: <http://trac.sagemath.org/ticket/17404#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
