#9107: Nested class name mangling can be wrong in case of double nesting
-------------------------------------+-------------------------------------
Reporter: hivert | Owner: nthiery
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: categories | Resolution:
Keywords: | Merged in:
Authors: Simon King | Reviewers: Volker Braun, Florent
Report Upstream: N/A | Hivert
Branch: | Work issues:
public/nested_class-9107 | Commit:
Dependencies: #12808 | 8b14e051c933f2ec2ae9891efbf329e7f0f2cba4
| Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:80 tscrim]:
> Here's a version with a patched version of the sphinx spkg.
Ah, that's nice: modifying standard spkgs is much simpler now that we have
a single repository. Thanks!
> Although I think your pull request is based on v1.2 (but it still
applied (for me) to our current v1.1). Could someone check to make sure I
created the patch correctly (and tell me what I should do instead if it
isn't right).
It seems to apply as it's supposed to (modulo trivial line offset).
I have made some minor improvement to the patch description.
I am a bit nervous about the removal of the former change log in SPKG.txt.
But if someone can confirm that this is the right thing to do, that's ok
for me.
Other than this, this sounds good to go!
Thanks again,
Nicolas
--
Ticket URL: <http://trac.sagemath.org/ticket/9107#comment:82>
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.
