Thanks! I needed something simple to learn how to review. Someone tell me if I did anything wrong.
Mariah On Jul 1, 8:52 am, David Kirkby <david.kir...@onetel.net> wrote: > If someone wants an easy patch to review, > > http://trac.sagemath.org/sage_trac/ticket/9399 > > should come pretty easy. Currently a header file is included on all > operating systems except Solaris (where _sun_ is defined). This just > removes the Sun specific bits and makes the header file behave no > differently on Solaris to any other platform. > > i.e. it changes > > #if defined(__sun) > typedef int int_fast32_t; > typedef long long int_fast64_t; > #else > #include <stdint.h> > #endif > > to > > #include <stdint.h> > > With this, Sage builds both on 32-bit and 64-bit. It does not work > fully on 64-bit, but by doing this, we remove the > > typedef long long int_fast64_t; > > which is invalid in 64-bit code, where int_fast64_t is a long, not a > long long. > > Dave -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
