Simplify remove_trailing_zeros

c8bd1e64 · Junekey Jeon · Victor Zverovich · 9b23e9dc · c8bd1e64
Commit c8bd1e64 authored Feb 13, 2022 by Junekey Jeon Committed by Victor Zverovich Feb 13, 2022
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 6 deletions

include/fmt/format-inl.h include/fmt/format-inl.h +3 -6

No files found.
--- a/include/fmt/format-inl.h
+++ b/include/fmt/format-inl.h
@@ -1843,12 +1843,9 @@ FMT_INLINE int remove_trailing_zeros(uint32_t& n) noexcept {
  int s = 0;
  while (true) {
    auto q = rotr(n * mod_inv_25, 2);
-    if (q <= std::numeric_limits<uint32_t>::max() / 100) {
+    if (q > std::numeric_limits<uint32_t>::max() / 100) break;
    n = q;
    s += 2;
-    } else {
-      break;
-    }
  }
  auto q = rotr(n * mod_inv_5, 1);
  if (q <= std::numeric_limits<uint32_t>::max() / 10) {