#4564: [with patch, with positive review] implement long long -> mpz_t
------------------------------+---------------------------------------------
Reporter: robertwb | Owner: somebody
Type: defect | Status: new
Priority: major | Milestone: sage-3.2.1
Component: basic arithmetic | Resolution:
Keywords: |
------------------------------+---------------------------------------------
Changes (by craigcitro):
* summary: [with patch, needs review] implement long long -> mpz_t =>
[with patch, with positive review] implement
long long -> mpz_t
Comment:
So the code here looks good. I even pulled out my old G4 and tested it on
a big-endian machine, and everything worked fine.
I do have one question, though: doesn't the `mpz_set_longlong` belong
somewhere more generic than `integer.pyx`? I would have put it somewhere
in libcsage. Of course, then it wouldn't be so easy to raise the exception
...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4564#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
