diff options
author | Ryan Harrison <rharrison@chromium.org> | 2017-09-27 11:07:51 -0400 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-09-27 16:18:11 +0000 |
commit | 8b1408e7053247def45659515cda19a64a7fccd5 (patch) | |
tree | f3dfb5ebd24fb4993c6b915917b97250b3739555 /xfa | |
parent | bacf75eeeb5d89fe60b2c77584350ca1b1432bd5 (diff) | |
download | pdfium-8b1408e7053247def45659515cda19a64a7fccd5.tar.xz |
Remove FXSYS_strlen and FXSYS_wcslenchromium/3226
With the conversion of internal string sizes to size_t, these wrappers
are no longer needed. Replacing them with strlen and wcslen
respectively.
BUG=pdfium:828
Change-Id: Ia087ca2ddaf688a57ec9bd9ddfb8533cbe41510d
Reviewed-on: https://pdfium-review.googlesource.com/14890
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'xfa')
-rw-r--r-- | xfa/fgas/font/cfgas_defaultfontmanager.cpp | 2 | ||||
-rw-r--r-- | xfa/fxfa/cxfa_ffwidget.cpp | 6 | ||||
-rw-r--r-- | xfa/fxfa/cxfa_textparser.cpp | 4 | ||||
-rw-r--r-- | xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp | 3 | ||||
-rw-r--r-- | xfa/fxfa/parser/cxfa_dataexporter.cpp | 2 | ||||
-rw-r--r-- | xfa/fxfa/parser/cxfa_itemlayoutprocessor.cpp | 2 |
6 files changed, 9 insertions, 10 deletions
diff --git a/xfa/fgas/font/cfgas_defaultfontmanager.cpp b/xfa/fgas/font/cfgas_defaultfontmanager.cpp index 4b9091f690..6f39797927 100644 --- a/xfa/fgas/font/cfgas_defaultfontmanager.cpp +++ b/xfa/fgas/font/cfgas_defaultfontmanager.cpp @@ -30,7 +30,7 @@ RetainPtr<CFGAS_GEFont> CFGAS_DefaultFontManager::GetFont( dwStyle |= FX_FONTSTYLE_Italic; const wchar_t* pReplace = pCurFont->pReplaceFont; - int32_t iLength = FXSYS_wcslen(pReplace); + int32_t iLength = wcslen(pReplace); while (iLength > 0) { const wchar_t* pNameText = pReplace; while (*pNameText != L',' && iLength > 0) { diff --git a/xfa/fxfa/cxfa_ffwidget.cpp b/xfa/fxfa/cxfa_ffwidget.cpp index 859287388f..77f1ac5006 100644 --- a/xfa/fxfa/cxfa_ffwidget.cpp +++ b/xfa/fxfa/cxfa_ffwidget.cpp @@ -1780,7 +1780,7 @@ static uint8_t* XFA_RemoveBase64Whitespace(const uint8_t* pStr, int32_t iLen) { uint8_t* pCP; int32_t i = 0, j = 0; if (iLen == 0) { - iLen = FXSYS_strlen((char*)pStr); + iLen = strlen((char*)pStr); } pCP = FX_Alloc(uint8_t, iLen + 1); for (; i < iLen; i++) { @@ -1799,11 +1799,11 @@ static int32_t XFA_Base64Decode(const char* pStr, uint8_t* pOutBuffer) { return 0; } uint8_t* pBuffer = - XFA_RemoveBase64Whitespace((uint8_t*)pStr, FXSYS_strlen((char*)pStr)); + XFA_RemoveBase64Whitespace((uint8_t*)pStr, strlen((char*)pStr)); if (!pBuffer) { return 0; } - int32_t iLen = FXSYS_strlen((char*)pBuffer); + int32_t iLen = strlen((char*)pBuffer); int32_t i = 0, j = 0; uint32_t dwLimb = 0; for (; i + 3 < iLen; i += 4) { diff --git a/xfa/fxfa/cxfa_textparser.cpp b/xfa/fxfa/cxfa_textparser.cpp index 9cbb72aa19..35e9b01580 100644 --- a/xfa/fxfa/cxfa_textparser.cpp +++ b/xfa/fxfa/cxfa_textparser.cpp @@ -84,8 +84,8 @@ std::unique_ptr<CFX_CSSStyleSheet> CXFA_TextParser::LoadDefaultSheetStyle() { L"sub{vertical-align:-15em;font-size:.66em}"; auto sheet = pdfium::MakeUnique<CFX_CSSStyleSheet>(); - return sheet->LoadBuffer(s_pStyle, FXSYS_wcslen(s_pStyle)) ? std::move(sheet) - : nullptr; + return sheet->LoadBuffer(s_pStyle, wcslen(s_pStyle)) ? std::move(sheet) + : nullptr; } RetainPtr<CFX_CSSComputedStyle> CXFA_TextParser::CreateRootStyle( diff --git a/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp b/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp index b4d070ac7d..00df0163f7 100644 --- a/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp +++ b/xfa/fxfa/fm2js/cxfa_fm2jscontext.cpp @@ -3342,8 +3342,7 @@ WideString CXFA_FM2JSContext::DecodeXML(const WideString& wsXMLString) { const wchar_t* const strName[] = {L"quot", L"amp", L"apos", L"lt", L"gt"}; int32_t iIndex = 0; while (iIndex < 5) { - if (memcmp(strString, strName[iIndex], FXSYS_wcslen(strName[iIndex])) == - 0) { + if (memcmp(strString, strName[iIndex], wcslen(strName[iIndex])) == 0) { break; } ++iIndex; diff --git a/xfa/fxfa/parser/cxfa_dataexporter.cpp b/xfa/fxfa/parser/cxfa_dataexporter.cpp index 5433ba788e..44b0dca98c 100644 --- a/xfa/fxfa/parser/cxfa_dataexporter.cpp +++ b/xfa/fxfa/parser/cxfa_dataexporter.cpp @@ -399,7 +399,7 @@ void XFA_DataExporter_RegenerateFormFile( pStream->WriteString(L" xmlns=\""); const wchar_t* pURI = XFA_GetPacketByIndex(XFA_PACKET_Form)->pURI; - pStream->WriteString(WideStringView(pURI, FXSYS_wcslen(pURI))); + pStream->WriteString(WideStringView(pURI, wcslen(pURI))); WideString wsVersionNumber; RecognizeXFAVersionNumber( diff --git a/xfa/fxfa/parser/cxfa_itemlayoutprocessor.cpp b/xfa/fxfa/parser/cxfa_itemlayoutprocessor.cpp index 7d9a2188c8..8bc71f11d3 100644 --- a/xfa/fxfa/parser/cxfa_itemlayoutprocessor.cpp +++ b/xfa/fxfa/parser/cxfa_itemlayoutprocessor.cpp @@ -36,7 +36,7 @@ std::vector<WideString> SeparateStringW(const wchar_t* pStr, if (!pStr) return ret; if (iStrLen < 0) - iStrLen = FXSYS_wcslen(pStr); + iStrLen = wcslen(pStr); const wchar_t* pToken = pStr; const wchar_t* pEnd = pStr + iStrLen; |