[ccan] [PATCH 1/3] opt: add integer helpers that accept k, M, G, T, P, E suffixes
Douglas Bagnall
douglas at paradise.net.nz
Sun Aug 14 15:40:04 EST 2011
On 14/08/11 00:41, Rusty Russell wrote:
>
> One nasty bug showed up in the final patch when testing on 32 bit:
Oh, sorry, and thank you.
I did afterwards realise that I'd never tried 32 bit, so I'm glad the
tests compiled and even tested something.
> I changed sizeof(suffixes) to strlen(suffixes) (rather than changing
> suffixes to an array), which is one less than you meant, but more
> correct.
I think the strlen is what I meant -- it is just, somehow, not what I
wrote.
Douglas
More information about the ccan
mailing list