#5970: Weak references in Polynomial Ring cache
--------------------------------------------+-------------------------------
Reporter: SimonKing | Owner: malb
Type: defect | Status: positive_review
Priority: critical | Milestone: sage-4.8
Component: commutative algebra | Keywords: polynomial ring
cache weak reference
Work_issues: regression for test_ec_leak? | Upstream: N/A
Reviewer: Simon King | Author:
Merged: | Dependencies:
--------------------------------------------+-------------------------------
Comment(by SimonKing):
Replying to [comment:32 zimmerma]:
> Simon, there is a {{{duplicate}}} field, which would be better than
{{{positive review}}}.
There is no duplicate field, because choosing the resolution of a ticket
is reserved to people with administrator rights. Apparently you have these
rights, but I haven't.
In addition, I am very much sure that Jeroen said that this is the correct
way to proceed: If one thinks that the ticket is a duplicate then you
review it accordingly, putting it as "positive review". Then the RELEASE
MANAGER (nobody else!!) closes the ticket by choosing the resolution
"duplicate", if he believes that the reasons given in the review make
sense.
> Also, maybe you might put this as {{{needs review}}}, asking someone
(maybe you) to check that your patch for #715 indeed solves the problem.
I can of course not review my own patch from #715. I can merely state that
the problem of the ticket here is in fact a sub-problem of ticket #715.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5970#comment:33>
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 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/sage-trac?hl=en.
