Bug in BigInt::FromWords64 on 32-bit?

[ClearScript Developers]

Greetings!

Consider the line here:

    if (kDigitBits == 32 && words[words64_count - 1] <= (1ULL << 32)) length--;

It seems like the goal here is to discard the final 32-bit "half-word" if it's zero. If that's correct, shouldn't the comparison operator be "<" rather than "<="?

Thanks!

[Jakob Kummerow]

Yes, that looks like a bug. I'll make a fix. Thanks for the report!