Remove a TODO related to C++14.
The amount of effort required is not worth it. A simple way to fulfill
the TODO with floor() and log10() does not work, because those functions
and their std:: equivalents are not constexpr.
Change-Id: I522aa9bf8f44c472ae85b500bd39678435d5c7a3
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/70735
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/fxjs/cjs_publicmethods.cpp b/fxjs/cjs_publicmethods.cpp
index 8ba2565..02dced8 100644
--- a/fxjs/cjs_publicmethods.cpp
+++ b/fxjs/cjs_publicmethods.cpp
@@ -800,8 +800,7 @@
// When the |iDec| value is too big, Acrobat will just return "%".
static constexpr int kDecLimit = 512;
- // TODO(thestig): Calculate this once C++14 can be used to declare variables
- // in constexpr functions.
+ // This count must be in sync with |kDecLimit|.
static constexpr size_t kDigitsInDecLimit = 3;
WideString& Value = pEvent->Value();
if (iDec > kDecLimit) {