Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
On Tue, Dec 3, 2013 at 9:26 PM, Alexander Nasonov 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 hadn't at tha

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 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 br

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
On Tue, Dec 3, 2013 at 8:15 PM, Lourival Vieira Neto wrote: > Hi Justin, > > On Tue, Dec 3, 2013 at 8:04 PM, Justin Cormack > wrote: >> >> On 3 Dec 2013 16:02, "Christos Zoulas" wrote: >>> >>> On Dec 3, 11:45am, ln...@netbsd.org (Lourival Vieira Neto) wrote: >>> -- Subject: Re: CVS commit: src >

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
Hi Justin, On Tue, Dec 3, 2013 at 8:04 PM, Justin Cormack wrote: > > On 3 Dec 2013 16:02, "Christos Zoulas" wrote: >> >> 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

Re: CVS commit: src/sys/arch/xen/xen

2013-12-03 Thread David Laight
On Tue, Dec 03, 2013 at 08:51:00PM +, Manuel Bouyer wrote: > Module Name: src > Committed By: bouyer > Date: Tue Dec 3 20:51:00 UTC 2013 > > Modified Files: > src/sys/arch/xen/xen: evtchn.c > > Log Message: > Remove the "evtchn_do_event: handler %p didn't lower ipl %d %d\n" pr

Re: CVS commit: src/sys/arch/xen/xen

2013-12-03 Thread Jeff Rizzo
On 12/3/13 12:51 PM, Manuel Bouyer wrote: Module Name:src Committed By: bouyer Date: Tue Dec 3 20:51:00 UTC 2013 Modified Files: src/sys/arch/xen/xen: evtchn.c Log Message: Remove the "evtchn_do_event: handler %p didn't lower ipl %d %d\n" printf. With help from Robert E

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
On Tue, Dec 3, 2013 at 2:05 PM, Christos Zoulas wrote: > In article > , > Lourival Vieira Neto wrote: >>Hi Valery, >> >>On Tue, Dec 3, 2013 at 11:58 AM, Valery Ushakov wrote: >>> On Tue, Dec 03, 2013 at 11:45:15 -0200, Lourival Vieira Neto wrote: >>> Also, moving to intmax_t, would help i

Re: CVS commit: src

2013-12-03 Thread Christos Zoulas
In article , Lourival Vieira Neto wrote: >Hi Valery, > >On Tue, Dec 3, 2013 at 11:58 AM, Valery Ushakov 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 (L

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 co

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
Hi Valery, On Tue, Dec 3, 2013 at 11:58 AM, Valery Ushakov 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 >> length modifi

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 Christos, On Tue, Dec 3, 2013 at 11:22 AM, Christos Zoulas wrote: > In article <20131203082610.ga21...@mail.duskware.de>, > Martin Husemann 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_ >>>

Re: CVS commit: src

2013-12-03 Thread Lourival Vieira Neto
Hi Martin, On Tue, Dec 3, 2013 at 6:26 AM, Martin Husemann 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 >> long' is a good

Re: CVS commit: src

2013-12-03 Thread Christos Zoulas
In article <20131203082610.ga21...@mail.duskware.de>, Martin Husemann 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 >> long'

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 wrote: > > > Module Name:src > > Committed By: joerg > > Date: Tue Dec 3 02:28:51 UTC 2013 > > > > Modified Files: > > src/lib/libc/compiler_rt: Makef

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 argume