diff options
author | Lei Zhang <thestig@chromium.org> | 2017-07-20 23:41:26 -0700 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-07-21 18:38:48 +0000 |
commit | d0856ba49c2bdcb288e0976d2162db17210a66df (patch) | |
tree | 48b8620b823fccaca15a6149dbb130cdc417bcbc /core/fpdfdoc/cpdf_nametree.cpp | |
parent | 926c65a49ca4769274022678e53144fe335c70b5 (diff) | |
download | pdfium-d0856ba49c2bdcb288e0976d2162db17210a66df.tar.xz |
Fix nits from commit 67ccef7.chromium/3164
Change-Id: I24b191ecfe8b65a7d0ddc4958dc117de9ed9ae83
Reviewed-on: https://pdfium-review.googlesource.com/8630
Reviewed-by: Jane Liu <janeliulwq@google.com>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdfdoc/cpdf_nametree.cpp')
-rw-r--r-- | core/fpdfdoc/cpdf_nametree.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/core/fpdfdoc/cpdf_nametree.cpp b/core/fpdfdoc/cpdf_nametree.cpp index 04cb1b9e40..f30a27c681 100644 --- a/core/fpdfdoc/cpdf_nametree.cpp +++ b/core/fpdfdoc/cpdf_nametree.cpp @@ -43,14 +43,14 @@ CPDF_Object* SearchNameNode(CPDF_Dictionary* pNode, for (size_t i = 0; i < dwCount; i++) { CFX_WideString csValue = pNames->GetUnicodeTextAt(i * 2); int32_t iCompare = csValue.Compare(csName); - if (iCompare <= 0) { - if (ppFind) - *ppFind = pNames; - if (iCompare < 0) - continue; - } else { + if (iCompare > 0) break; - } + + if (ppFind) + *ppFind = pNames; + if (iCompare < 0) + continue; + nIndex += i; return pNames->GetDirectObjectAt(i * 2 + 1); } @@ -170,7 +170,7 @@ int CPDF_NameTree::GetIndex(const CFX_WideString& csName) const { CPDF_Object* CPDF_NameTree::LookupValueAndName(int nIndex, CFX_WideString* csName) const { - *csName = CFX_WideString(); + csName->clear(); if (!m_pRoot) return nullptr; |