Re: CVS commit: src

2013-12-03 Thread Martin Husemann
On Mon, Dec 02, 2013 at 10:33:05PM -0200, Lourival Vieira Neto wrote: I also have no problem to step back and use 'long long', if _we_ choose to reconsider that. IMHO, the fact that Lua 5.3 is using 'long long' is a good argument for that. I do prefer explicit width type, but my main argument

Re: CVS commit: src/lib/libc/compiler_rt

2013-12-03 Thread Joerg Sonnenberger
On Mon, Dec 02, 2013 at 11:10:43PM -0800, Matt Thomas wrote: On Dec 2, 2013, at 6:28 PM, Joerg Sonnenberger jo...@netbsd.org wrote: Module Name:src Committed By: joerg Date: Tue Dec 3 02:28:51 UTC 2013 Modified Files: src/lib/libc/compiler_rt:

Re: CVS commit: src

2013-12-03 Thread Christos Zoulas
In article 20131203082610.ga21...@mail.duskware.de, Martin Husemann mar...@duskware.de wrote: On Mon, Dec 02, 2013 at 10:33:05PM -0200, Lourival Vieira Neto wrote: I also have no problem to step back and use 'long long', if _we_ choose to reconsider that. IMHO, the fact that Lua 5.3 is using

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
Hi Martin, On Tue, Dec 3, 2013 at 6:26 AM, Martin Husemann mar...@duskware.de wrote: On Mon, Dec 02, 2013 at 10:33:05PM -0200, Lourival Vieira Neto wrote: I also have no problem to step back and use 'long long', if _we_ choose to reconsider that. IMHO, the fact that Lua 5.3 is using 'long

Re: CVS commit: src

2013-12-03 Thread Valery Ushakov
On Tue, Dec 03, 2013 at 11:45:15 -0200, Lourival Vieira Neto wrote: Also, moving to intmax_t, would help in string library. It needs a length modifier for string.format (LUA_INTFRMLEN). AFAIK, there is no length modifier defined for int64_t. Using intmax_t we could just use j. There is:

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
Hi Valery, On Tue, Dec 3, 2013 at 11:58 AM, Valery Ushakov u...@stderr.spb.ru wrote: On Tue, Dec 03, 2013 at 11:45:15 -0200, Lourival Vieira Neto wrote: Also, moving to intmax_t, would help in string library. It needs a length modifier for string.format (LUA_INTFRMLEN). AFAIK, there is no

Re: CVS commit: src

2013-12-03 Thread Christos Zoulas
On Dec 3, 11:45am, ln...@netbsd.org (Lourival Vieira Neto) wrote: -- Subject: Re: CVS commit: src | Also, moving to intmax_t, would help in string library. It needs a | length modifier for string.format (LUA_INTFRMLEN). AFAIK, there is no | length modifier defined for int64_t. Using intmax_t we

Re: CVS commit: src

2013-12-03 Thread Christos Zoulas
In article calhokcbno1cgv067xxzzp249p+moa21ftqyfqkuvrpnjqzg...@mail.gmail.com, Lourival Vieira Neto ln...@netbsd.org wrote: Hi Valery, On Tue, Dec 3, 2013 at 11:58 AM, Valery Ushakov u...@stderr.spb.ru wrote: On Tue, Dec 03, 2013 at 11:45:15 -0200, Lourival Vieira Neto wrote: Also, moving to

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
Hi Justin, On Tue, Dec 3, 2013 at 8:04 PM, Justin Cormack jus...@specialbusservice.com wrote: On 3 Dec 2013 16:02, Christos Zoulas chris...@zoulas.com wrote: On Dec 3, 11:45am, ln...@netbsd.org (Lourival Vieira Neto) wrote: -- Subject: Re: CVS commit: src | Also, moving to intmax_t, would

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
On Tue, Dec 3, 2013 at 8:15 PM, Lourival Vieira Neto ln...@netbsd.org wrote: Hi Justin, On Tue, Dec 3, 2013 at 8:04 PM, Justin Cormack jus...@specialbusservice.com wrote: On 3 Dec 2013 16:02, Christos Zoulas chris...@zoulas.com wrote: On Dec 3, 11:45am, ln...@netbsd.org (Lourival Vieira

Re: CVS commit: src

2013-12-03 Thread Christos Zoulas
On Dec 3, 10:04pm, jus...@specialbusservice.com (Justin Cormack) wrote: -- Subject: Re: CVS commit: src | No that is a bad side effect. It must always be 64 bits. In the kernel you | have to deal with uint64_t which will behave differently if intmax_t is | ever bigger than 64 bits, so code will

Re: CVS commit: src

2013-12-03 Thread Alexander Nasonov
Lourival Vieira Neto wrote: I wasn't in that thread at that time. However, I'll carefully read it. If you haven't done so yet, it's a good idea to subscribe to source-changes-d@. Anyway, I think that the missing implementation of luai_numpow() doesn't break anything. Do you have a test to

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
On Tue, Dec 3, 2013 at 9:26 PM, Alexander Nasonov al...@yandex.ru wrote: Lourival Vieira Neto wrote: I wasn't in that thread at that time. However, I'll carefully read it. If you haven't done so yet, it's a good idea to subscribe to source-changes-d@. Yes, I subscribed already =). I just