Fixed potential bug with -MINIMUM_VALUE

Original code had something like `N < 0 ? -N : N` which could fail if `N` has minimum value of appropriate type. For example, in original code this could fail:
```cpp
const long long int LL_MIN = std::numeric_limits<long long int>::min(); //probably: -9223372036854775808
static const char *number = to_string<LL_MIN, 10>;
```

Also note that this code relies on "modulo on negative first operands" which was implementation defined in C++03 and before, but with C++11 and later it is defined by standard (see `ISO14882:2011(e)`), and this code anyway targets C++17 so we're safe here.
pull/1/head
EnilPajic 4 years ago committed by GitHub
parent 2be778931d
commit 5baf5d3dae
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -23,7 +23,7 @@ struct to_string_t {
// fits to the number perfectly. // fits to the number perfectly.
char buf[([] { char buf[([] {
unsigned int len = N >= 0 ? 1 : 2; unsigned int len = N >= 0 ? 1 : 2;
for (auto n = N < 0 ? -N : N; n; len++, n /= base); for (auto n = N; n; len++, n /= base);
return len; return len;
}())]; }())];
@ -33,8 +33,8 @@ struct to_string_t {
constexpr to_string_t() { constexpr to_string_t() {
auto ptr = buf + sizeof(buf) / sizeof(buf[0]); auto ptr = buf + sizeof(buf) / sizeof(buf[0]);
*--ptr = '\0'; *--ptr = '\0';
for (auto n = N < 0 ? -N : N; n; n /= base) for (auto n = N; n; n /= base)
*--ptr = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ"[n % base]; *--ptr = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ"[(N < 0 ? -1 : 1) * (n % base)];
if (N < 0) if (N < 0)
*--ptr = '-'; *--ptr = '-';
} }

Loading…
Cancel
Save