
3 Mar
2010
3 Mar
'10
5:14 p.m.
On Tue, 02 Mar 2010 16:05 +0000, "Sylvester-Bradley, Gareth" <Gareth.Sylvester-Bradley@eu.sony.com> wrote:
This is definitely a mistake, since it means only about 4 billion 'til a collision, completely destroying the utility of random UUIDs. Not to mention that shifting by >= the width isn't well-defined, which means there's no guarantee it even gives different UUIDs.
Right.
Good catch.
Absolutely great catch! It is definitely a mistake.
Simplest patch could be just:
370a370
i=0;
This is exactly what I meant it to be. Reset i every time it equals sizeof(unsigned long). Just to double check, I fix this in trunk and wait to put it into release until it is open for these changes. Correct?
Best regards, Gareth
Thanks very much! Andy.