From 768caf7c9a42da502e2f0cd393816f502bbb3776 Mon Sep 17 00:00:00 2001 From: Roman Koshelev Date: Sun, 12 Sep 2021 18:09:05 +0300 Subject: [PATCH] Removed redundant format_decimal implementation for constexpr context --- include/fmt/format.h | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/include/fmt/format.h b/include/fmt/format.h index b2bae65a..546f6b55 100644 --- a/include/fmt/format.h +++ b/include/fmt/format.h @@ -1090,19 +1090,11 @@ template struct format_decimal_result { // buffer of specified size. The caller must ensure that the buffer is large // enough. template -FMT_CONSTEXPR20 auto format_decimal(Char* out, UInt value, int size) +FMT_CONSTEXPR auto format_decimal(Char* out, UInt value, int size) -> format_decimal_result { FMT_ASSERT(size >= count_digits(value), "invalid digit count"); out += size; Char* end = out; - if (is_constant_evaluated()) { - while (value >= 10) { - *--out = static_cast('0' + value % 10); - value /= 10; - } - *--out = static_cast('0' + value); - return {out, end}; - } while (value >= 100) { // Integer division is slow so do it for a group of two digits instead // of for every digit. The idea comes from the talk by Alexandrescu