#11734: sage_wraps should only read the sources of wrapped functions when 
needed.
-------------------------+--------------------------------------------------
   Reporter:  SimonKing  |          Owner:  jason                               
 
       Type:  defect     |         Status:  needs_review                        
 
   Priority:  blocker    |      Milestone:  sage-4.7.2                          
 
  Component:  misc       |       Keywords:  sage_wraps sources gentoo 
startuptime
Work_issues:             |       Upstream:  N/A                                 
 
   Reviewer:             |         Author:  Simon King                          
 
     Merged:             |   Dependencies:                                      
 
-------------------------+--------------------------------------------------

Comment(by SimonKing):

 Replying to [comment:3 saraedum]:
 > I do not fully understand your patch. How does it differ from simply
 taking out the call to sage_getargspec and moving it into a lambda? (see
 my attached patch)

 If I remember correctly, I once tried that approach. The line `argspec =
 sage_getargspec(wrapped)` was introduced for a reason (something went
 wrong), but I can't remember what it was.

 Did you run full tests with your patch?

 > Btw. On my install of sage-4.7.2alpha2 the doctest described in
 ''Bugfix'' does not fail.

 Really? Strange. It failed for me both with sage-4.7.2.alpha1 and with
 sage-4.6.2.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11734#comment:4>
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.

Reply via email to