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 --- testing/test_support.h | 4 ++-- testing/unit_test_main.cpp | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'testing') 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