#17019: preparser for .sage files no longer respects module docstrings
-------------------------------------+-------------------------------------
Reporter: jsrn | Owner:
Type: defect | Status: needs_info
Priority: minor | Milestone: sage-6.4
Component: misc | Resolution:
Keywords: docstring, | Merged in:
preparse | Reviewers: Jeroen Demeyer
Authors: Johan S. R. | Work issues:
Nielsen | Commit:
Report Upstream: N/A | 86350b34b6c40240becb3a4adfc9025fbaba4386
Branch: | Stopgaps:
u/jdemeyer/ticket/17019 |
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by jsrn):
* status: needs_work => needs_info
* branch: u/jsrn/ticket/17019 => u/jdemeyer/ticket/17019
Comment:
No need to get snappy -- obviously the problem was there or I wouldn't
waste my time on this. It seems, however, that somewhere between 6.2 and
6.3 the generation of the _sage_const_X lines were moved, and this
presents an alternative solution to the problem (those lines were
previously put before the module doc-string).
Now for this ticket: the change in sage-preparse could be rolled back
since it is not necessary for the moment. The doc-test changes could be
retained since it actually tests something which was not tested before.
The latest test that you saw was however rather silly, since I forgot to
reinstate a constant in the code after I merged the two doctests into one.
I've fixed the doc test...
--
Ticket URL: <http://trac.sagemath.org/ticket/17019#comment:21>
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.
