#17819: Use uintmax_t for Integer.divisors
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.6
Component: c_lib | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/ticket/17819 | 3bc79b71b49aa37b7307174f29e228a6c2bc81f6
Dependencies: #17794, #10257 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by fbissey):
Replying to [comment:19 jdemeyer]:
> Replying to [comment:18 fbissey]:
> > I would favor portability but that may be a discussion we need to have
on sage-devel.
> What do you think of my proposal to use `unsigned long`? That would be
portable and clean. On 64-bit there would be no difference.
>
> For small inputs, using `unsigned long` will even be faster on 32-bit
systems, since the compiler wouldn't need to emulate 64-bit `long long`
operations. For larger inputs, there might be a slow-down but it might not
be so bad given that there are no native 64-bit operations anyway.
I am OK with using `unsigned long` but it all gave me food for thought.
--
Ticket URL: <http://trac.sagemath.org/ticket/17819#comment:20>
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/d/optout.
