Re: pgsql: Fix pg_size_bytes() to be more portable.
От | Dean Rasheed |
---|---|
Тема | Re: pgsql: Fix pg_size_bytes() to be more portable. |
Дата | |
Msg-id | CAEZATCWb+PEpsPi1w4MnnVSSs_y6OQJULRkjgXaj72ugOtLR7w@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: pgsql: Fix pg_size_bytes() to be more portable. (Tom Lane <tgl@sss.pgh.pa.us>) |
Ответы |
Re: pgsql: Fix pg_size_bytes() to be more portable.
|
Список | pgsql-committers |
On 20 February 2016 at 14:54, Tom Lane <tgl@sss.pgh.pa.us> wrote: > Dean Rasheed <dean.a.rasheed@gmail.com> writes: >> Fix pg_size_bytes() to be more portable. >> Commit 53874c5228fe16589a4d01b3e1fab3678e0fd8e3 broke various 32-bit >> buildfarm machines because it incorrectly used an 'L' suffix for what >> needed to be a 64-bit literal. Thanks to Michael Paquier for helping >> to diagnose this. > > That's still not right: not all compilers support "long long", and the > ones that don't won't have "LL" notation either. > > Project style is to use something like "(uint64) 1024" instead. > OK, will fix. BTW, I found a couple of instances of 'LL' in ecpglib/prepare.c, which is why I thought it was OK to use it. Regards, Dean
В списке pgsql-committers по дате отправления: