Fix some sign comparison warnings.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1279493003 .
(cherry picked from commit b4c2167fff88665089eeab47afa45a9e890060a3)
Review URL: https://codereview.chromium.org/1287703002 .
diff --git a/fpdfsdk/src/javascript/PublicMethods.cpp b/fpdfsdk/src/javascript/PublicMethods.cpp
index 8d7d207..f71ea37 100644
--- a/fpdfsdk/src/javascript/PublicMethods.cpp
+++ b/fpdfsdk/src/javascript/PublicMethods.cpp
@@ -1135,8 +1135,8 @@
if (-1 != pEvent->SelStart())
w_strSelected = w_strValue2.substr(pEvent->SelStart(),
(pEvent->SelEnd() - pEvent->SelStart()));
- FX_BOOL bHasSign =
- (w_strValue2.find('-') != -1) && (w_strSelected.find('-') == -1);
+ bool bHasSign = (w_strValue2.find('-') != std::wstring::npos) &&
+ (w_strSelected.find('-') == std::wstring::npos);
if (bHasSign) {
// can't insert "change" in front to sign postion.
if (pEvent->SelStart() == 0) {
@@ -1159,7 +1159,7 @@
break;
}
- FX_BOOL bHasSep = (w_strValue2.find(cSep) != -1);
+ bool bHasSep = (w_strValue2.find(cSep) != std::wstring::npos);
for (std::wstring::iterator it = w_strChange2.begin();
it != w_strChange2.end(); it++) {
if (*it == cSep) {