From 8b36e5cc3d5f5f579c6b060e2c40b896a4b02bc0 Mon Sep 17 00:00:00 2001 From: tsepez Date: Fri, 8 Apr 2016 09:00:35 -0700 Subject: Make CFX_WideString::FromLocal() take a CFX_ByteStringC arg It doesn't persist the string beyond the duration of the call, hence it should take the *StringC variant. Doing so avoids some allocs by changing to the *StringC ctor in a few places, at the cost of some explicit .ToByteStringC() calls (which are cheap). Review URL: https://codereview.chromium.org/1862953004 --- fpdfsdk/javascript/PublicMethods.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'fpdfsdk/javascript/PublicMethods.cpp') diff --git a/fpdfsdk/javascript/PublicMethods.cpp b/fpdfsdk/javascript/PublicMethods.cpp index 308163d454..d5b961d61d 100644 --- a/fpdfsdk/javascript/PublicMethods.cpp +++ b/fpdfsdk/javascript/PublicMethods.cpp @@ -837,7 +837,7 @@ FX_BOOL CJS_PublicMethods::AFNumber_Format(IJS_Context* cc, } // for processing currency string - Value = CFX_WideString::FromLocal(strValue); + Value = CFX_WideString::FromLocal(strValue.AsByteStringC()); std::wstring strValue2 = Value.c_str(); if (bCurrencyPrepend) @@ -1120,7 +1120,7 @@ FX_BOOL CJS_PublicMethods::AFPercent_Format( if (iNegative) strValue = "-" + strValue; strValue += "%"; - Value = CFX_WideString::FromLocal(strValue); + Value = CFX_WideString::FromLocal(strValue.AsByteStringC()); #endif return TRUE; } -- cgit v1.2.3