#14818: Declare PARI finite field functions (FF_*), wrap ffgen() and ffinit()
--------------------------------+-------------------------------------------
       Reporter:  pbruin        |         Owner:  cpernet          
           Type:  enhancement   |        Status:  needs_work       
       Priority:  minor         |     Milestone:  sage-5.12        
      Component:  finite rings  |    Resolution:                   
       Keywords:  pari          |   Work issues:  doctests         
Report Upstream:  N/A           |     Reviewers:  Jean-Pierre Flori
        Authors:  Peter Bruin   |     Merged in:                   
   Dependencies:                |      Stopgaps:                   
--------------------------------+-------------------------------------------

Comment (by jpflori):

 There is also a problem with the docstring for ffinit which mentions a
 polynomial P.

 And the doc avout the v variable is kind of obscure.
 In the docstring for get_var it's mentioned it should be a string or -1.
 Here it seems you suggest it can be an integer.
 Indeed, the PARI ffinit functions wants a long and will treat negative
 input in a special way, but the point of the call to get_var is to convert
 the var's string to the correct integer.
 So either skip the get_var call or correcte the docstring.
 I think the second option is better, and is what you wanted, isn't it?

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14818#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 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/groups/opt_out.


Reply via email to