From fed39cf4a23341cf9cb5a5b432248b4247022282 Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Wed, 1 Mar 2017 14:24:03 -0500 Subject: Fix font manager negation issue In https://codereview.chromium.org/2559903002 we refactored the font manager in XFA but got part of the negation wrong. Fixup logic. Change-Id: Idd51ff846fe64a56908ec26bf2798c1df8433611 Reviewed-on: https://pdfium-review.googlesource.com/2891 Reviewed-by: Tom Sepez Commit-Queue: dsinclair --- xfa/fgas/font/cfgas_fontmgr.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xfa/fgas/font/cfgas_fontmgr.cpp b/xfa/fgas/font/cfgas_fontmgr.cpp index bf243a49c6..57b499fb10 100644 --- a/xfa/fgas/font/cfgas_fontmgr.cpp +++ b/xfa/fgas/font/cfgas_fontmgr.cpp @@ -516,7 +516,7 @@ CFX_ByteString CFX_FontSourceEnum_File::GetNextFile() { FX_CloseFolder(pCurHandle); if (!m_FolderQueue.empty()) m_FolderQueue.pop_back(); - if (!m_FolderQueue.empty()) { + if (m_FolderQueue.empty()) { if (!m_FolderPaths.empty()) m_FolderPaths.pop_back(); return !m_FolderPaths.empty() ? GetNextFile() : ""; -- cgit v1.2.3