[Peek] NumberOfDigits(int.MinValue) throws unhandled OverflowException

[crutkas]

Bug

Math.Abs(int.MinValue) throws OverflowException. Edge case crash path.
Found by AI test review fleet

[daverayment]

I don't think this is correct.

NumberOfDigits() is called by GetPrecision() which is called by BytesToReadableString(). The latter is called with a ulong bytes parameter.

The way bytes is transformed is to divide it by Math.Pow(PowerFactor, index), which should give us a result between 1 and 1023, which is then further processed into a 0 to 999 value if it is greater than or equal to 1000.

So GetPrecision() (and then NumberOfDigits()) cannot be called with a negative value and the original code is correct as-is.