commit 067011d41d9073699d0682c55690a50c94097c93
author Lei Zhang <thestig@chromium.org> Wed Feb 05 13:52:47 2025 -0800
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Feb 05 13:52:47 2025 -0800
tree 0603f0191dfaffd44ef00bc836ca15bc15b38a64
parent 100e24682a0418b7d9efc5732c47d31d19ac90f4

Switch one FXSYS_wcstof() caller to StringToFloat()

Since FXSYS_wcstof() and StringToFloat() are basically the same, switch
the FXSYS_wcstof() caller that does not care about the amount of string
parsed to use StringToFloat().

Change-Id: I18fdccef5bb49dc402979705b0ae6f8568d86fbf
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/128451
Reviewed-by: Thomas Sepez <tsepez@google.com>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

fxjs/xfa/cjx_object.cpp

diff --git a/fxjs/xfa/cjx_object.cpp b/fxjs/xfa/cjx_object.cpp
index 4b672a3..bf11ad8 100644
--- a/fxjs/xfa/cjx_object.cpp
+++ b/fxjs/xfa/cjx_object.cpp
@@ -239,8 +239,7 @@
       SetBoolean(eAttr, !wsValue.EqualsASCII("0"), bNotify);
       break;
     case XFA_AttributeType::Integer:
-      SetInteger(eAttr,
-                 FXSYS_roundf(FXSYS_wcstof(wsValue.AsStringView(), nullptr)),
+      SetInteger(eAttr, FXSYS_roundf(StringToFloat(wsValue.AsStringView())),
                  bNotify);
       break;
     case XFA_AttributeType::Measure: