From d60d7cb936ea50a0dfab26372f3a8958f84250e7 Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Fri, 1 Jun 2018 15:28:54 +0000 Subject: Fix XFA guards These guards are using #if, when they should be using #ifdef. Caught using Clang's static analyzer. Change-Id: I86e99c5e4142cf84fff5f2365ad534f09ae40511 Reviewed-on: https://pdfium-review.googlesource.com/33510 Reviewed-by: Henrique Nakashima Commit-Queue: Ryan Harrison --- core/fxcrt/fx_bidi.h | 2 +- fxjs/cjs_document.cpp | 2 +- testing/test_support.h | 4 ++-- testing/unit_test_main.cpp | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/core/fxcrt/fx_bidi.h b/core/fxcrt/fx_bidi.h index 54d5255e3e..79a2ddd417 100644 --- a/core/fxcrt/fx_bidi.h +++ b/core/fxcrt/fx_bidi.h @@ -72,7 +72,7 @@ class CFX_BidiString { CFX_BidiChar::Direction m_eOverallDirection = CFX_BidiChar::LEFT; }; -#if PDF_ENABLE_XFA +#ifdef PDF_ENABLE_XFA void FX_BidiLine(std::vector* chars, size_t iCount); #endif // PDF_ENABLE_XFA diff --git a/fxjs/cjs_document.cpp b/fxjs/cjs_document.cpp index 24be12d370..e087e0eda9 100644 --- a/fxjs/cjs_document.cpp +++ b/fxjs/cjs_document.cpp @@ -450,7 +450,7 @@ CJS_Return CJS_Document::removeField( // and creating a new pageview at this point will cause bad things. CPDFSDK_PageView* pPageView = m_pFormFillEnv->GetPageView(pPage, false); if (pPageView) { -#if PDF_ENABLE_XFA +#ifdef PDF_ENABLE_XFA pPageView->DeleteAnnot(pWidget); #endif // PDF_ENABLE_XFA pPageView->UpdateRects(aRefresh); diff --git a/testing/test_support.h b/testing/test_support.h index f26e161544..56b2f113b3 100644 --- a/testing/test_support.h +++ b/testing/test_support.h @@ -13,7 +13,7 @@ #include "public/fpdfview.h" -#if PDF_ENABLE_XFA +#ifdef PDF_ENABLE_XFA #include "xfa/fgas/font/cfgas_fontmgr.h" #endif // PDF_ENABLE_XFA @@ -118,7 +118,7 @@ class TestLoader { const size_t m_Len; }; -#if PDF_ENABLE_XFA +#ifdef PDF_ENABLE_XFA CFGAS_FontMgr* GetGlobalFontManager(); #endif // PDF_ENABLE_XFA diff --git a/testing/unit_test_main.cpp b/testing/unit_test_main.cpp index 92dceefdb6..70eb51a3a1 100644 --- a/testing/unit_test_main.cpp +++ b/testing/unit_test_main.cpp @@ -15,7 +15,7 @@ #include "v8/include/v8.h" #endif // PDF_ENABLE_V8 -#if PDF_ENABLE_XFA +#ifdef PDF_ENABLE_XFA #include "core/fxge/cfx_fontmgr.h" #include "core/fxge/cfx_gemodule.h" #include "xfa/fgas/font/cfgas_fontmgr.h" @@ -74,7 +74,7 @@ int main(int argc, char** argv) { #endif // V8_USE_EXTERNAL_STARTUP_DATA #endif // PDF_ENABLE_V8 -#if PDF_ENABLE_XFA +#ifdef PDF_ENABLE_XFA env_ = new Environment(); // The env will be deleted by gtest. AddGlobalTestEnvironment(env_); -- cgit v1.2.3