tech-kern archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: [patch] changing lua_Number to int64_t
Lourival Vieira Neto wrote:
> On Sat, Nov 16, 2013 at 8:52 PM, Christos Zoulas
> <christos%astron.com@localhost> wrote:
> > In article <52872B0C.5080309%msys.ch@localhost>, Marc Balmer
> > <marc%msys.ch@localhost> wrote:
> >>Changing the number type to int64_t is certainly a good idea. Two
> >>questions, however:
> >
> > Why not intmax_t?
>
> My only argument is that int64_t has a well-defined width and, AFAIK,
> intmax_t could vary. But I have no strong feelings about this. Do you
> think intmax_t would be better?
int64_t should be enough to cover a range of exactly representable
integers in userspace Lua program where lua_Number is double.
I don't see a need for bigger type unless mainstream Lua switches to
long double. I don't expect it to happen any time soon.
PS Why do you still use a shadow copy of luaconf.h? Please add your
changes to the main luaconf.h. If you guard your kernel changes properly
with _KERNEL, they will not affect userspace.
PPS "%"PRId64 may break in C++11, space between the literals should fix it.
Alex
Home |
Main Index |
Thread Index |
Old Index