> On 23 Dec 2016, at 19:07, thierry.fourn...@arpalert.org wrote: > > Ok, thanks Willy. > > The news path in attachment. David, can you test the FreeBSD build ? > The patch is tested and validated for Linux.
Yes, it does fix FreeBSD build. > > Thierry > > > On Fri, 23 Dec 2016 14:50:38 +0100 > Willy Tarreau <w...@1wt.eu> wrote: > >> On Fri, Dec 23, 2016 at 02:37:13PM +0100, thierry.fourn...@arpalert.org >> wrote: >>> thanks Willy for the idea. I will write a patch ASAP, but. why a 32bits >>> cast and not a 64 bit cast ? >> >> First because existing code uses this already and it works. Second because >> the 64-bit check might be more expensive for 32-bit platforms than the >> double 32-bit check is for 64-bit platforms (though that's still to be >> verified in the assembly code, as some compilers manage to assign register >> pairs correctly). >> >> Willy >> > <0001-BUILD-lua-build-failed-on-FreeBSD.patch>
