Opened 12 years ago

Closed 12 years ago

#9898 closed enhancement (fixed)

Clean up and add functions to sage/libs/pari/decl.pxi

Reported by: Jeroen Demeyer Owned by: William Stein
Priority: major Milestone: sage-4.6
Component: interfaces Keywords:
Cc: Leif Leonhardy Merged in: sage-4.6.alpha2
Authors: Jeroen Demeyer Reviewers: Leif Leonhardy
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Leif Leonhardy)

  • Adds a file sage/libs/pari/declinl.pxi for declarations of inline functions (which is included by sage/libs/pari/decl.pxi).
  • Some clean up of sage/libs/pari/decl.pxi, in particular removing duplicate functions.
  • Adds/updates authors sections.

Attachments (2)

9898_pari_decl.patch (46.8 KB) - added by Jeroen Demeyer 12 years ago.
9898_authors.patch (1.9 KB) - added by Jeroen Demeyer 12 years ago.
Adds authors to files in sage/libs/pari, apply on top of 9898_pari_decl.patch

Download all attachments as: .zip

Change History (15)

comment:1 Changed 12 years ago by Jeroen Demeyer

Status: newneeds_review

comment:2 Changed 12 years ago by Leif Leonhardy

Cc: Leif Leonhardy added

comment:3 Changed 12 years ago by Leif Leonhardy

s/seperate/separate/

s/This files/This file/

comment:4 in reply to:  3 Changed 12 years ago by Jeroen Demeyer

Replying to leif:

s/seperate/separate/

s/This files/This file/

Done.

comment:5 Changed 12 years ago by Leif Leonhardy

Did you upload the wrong patch?

I just noticed the typos are back...

comment:6 Changed 12 years ago by Leif Leonhardy

The global t0 in gequal_long() is superfluous.

Changed 12 years ago by Jeroen Demeyer

Attachment: 9898_pari_decl.patch added

comment:7 Changed 12 years ago by Jeroen Demeyer

Authors: Jeroen Demeyer

Done.

comment:8 Changed 12 years ago by Leif Leonhardy

Reviewers: Leif Leonhardy
Status: needs_reviewpositive_review

Ok, I've now also tested this with Sage 4.6.alpha1 and #9876 (PARI 2.4.3.svn-12577.p7) on Ubuntu 9.04 x86 and Ubuntu 10.04 x86_64 (both ptestlong).

Patch is up-to-date, so positive review.

I've (of course?) not checked if all functions really come from the PARI source files they're claimed to come from. ;-)

It's up to you to convince Mitesh to merge it into 4.6.alpha2. :)

comment:9 Changed 12 years ago by Jeroen Demeyer

Description: modified (diff)

comment:10 Changed 12 years ago by Leif Leonhardy

Perhaps one should mention that pari/declinl.pxi gets included by pari/decl.pxi.

Add svn snapshot number?

Changed 12 years ago by Jeroen Demeyer

Attachment: 9898_authors.patch added

Adds authors to files in sage/libs/pari, apply on top of 9898_pari_decl.patch

comment:11 in reply to:  10 ; Changed 12 years ago by Jeroen Demeyer

Replying to leif:

Perhaps one should mention that pari/declinl.pxi gets included by pari/decl.pxi.

Done.

Add svn snapshot number?

I don't think that is so relevant (those files would not look that much different for other SVN snapshot numbers). Besides, people can still look at the ticket #9343 for more information.

comment:12 in reply to:  11 Changed 12 years ago by Leif Leonhardy

Description: modified (diff)

Replying to jdemeyer:

[...] people can still look at the ticket #9343 for more information.

:-)

Btw, is 08-15 a 'symbolic' date?

Positive review for the second patch, too.

comment:13 Changed 12 years ago by Mitesh Patel

Merged in: sage-4.6.alpha2
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.