Re: pgsql: Add pg_size_bytes() to parse human-readable size strings.

On 20 February 2016 at 10:33, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
> On Sat, Feb 20, 2016 at 7:17 PM, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> wrote:
>> On 20 February 2016 at 10:12, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>>> Happy first commit.
>>
>> Arg. Not so much.
>>
>> Looks like I broke something -- looking into it now :-(
>
> The terabyte conversion is at fault:
> Expected:
> ! -1tb | -1099511627776
> Result:
> ! -1tb | -1
>
> + else if (pg_strcasecmp(strptr, "gb") == 0)
> + multiplier = 1024 * 1024 * 1024;
> + else if (pg_strcasecmp(strptr, "tb") == 0)
> + multiplier = 1024 * 1024 * 1024 * 1024L;
> Why adding an 'L' here?

Ah, looks like it needs to be 'LL' because it needs to be a 64-bit literal.

Regards,
Dean