From c9297e896c697b5bb9c3b9e66791a834f5552170 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Tue, 10 Jul 2018 01:30:44 +0000 Subject: Fix duplicate symbols in Chromium builds. Commit 43f6bc80 copied over Chromium's base/debug/alias.cc, but that confuses Windows builds and Linux jumbo builds in Chromium. To fix this, wrap PDFium's copy in the pdfium namespace. BUG=chromium:860850 TBR=dsinclair@chromium.org Change-Id: I1ceec28b9ce6c2893bf030a5b6564dce6d6ec376 Reviewed-on: https://pdfium-review.googlesource.com/37430 Commit-Queue: Lei Zhang Reviewed-by: Lei Zhang --- core/fxcrt/fx_memory.cpp | 2 +- third_party/base/debug/alias.cc | 2 ++ third_party/base/debug/alias.h | 8 +++++--- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/core/fxcrt/fx_memory.cpp b/core/fxcrt/fx_memory.cpp index 4c7b36855c..c0ca7a02df 100644 --- a/core/fxcrt/fx_memory.cpp +++ b/core/fxcrt/fx_memory.cpp @@ -49,7 +49,7 @@ NEVER_INLINE void FX_OutOfMemoryTerminate() { // Convince the linker this should not be folded with similar functions using // Identical Code Folding. static int make_this_function_aliased = 0xbd; - base::debug::Alias(&make_this_function_aliased); + pdfium::base::debug::Alias(&make_this_function_aliased); // Termimate cleanly if we can, else crash at a specific address (0xbd). abort(); diff --git a/third_party/base/debug/alias.cc b/third_party/base/debug/alias.cc index 3ab554fe48..6ee2ee9752 100644 --- a/third_party/base/debug/alias.cc +++ b/third_party/base/debug/alias.cc @@ -6,6 +6,7 @@ #include "build/build_config.h" +namespace pdfium { namespace base { namespace debug { @@ -26,3 +27,4 @@ void Alias(const void* var) { } // namespace debug } // namespace base +} // namespace pdfium diff --git a/third_party/base/debug/alias.h b/third_party/base/debug/alias.h index 4d02e7af74..4fac6667b3 100644 --- a/third_party/base/debug/alias.h +++ b/third_party/base/debug/alias.h @@ -2,9 +2,10 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -#ifndef BASE_DEBUG_ALIAS_H_ -#define BASE_DEBUG_ALIAS_H_ +#ifndef PDFIUM_THIRD_PARTY_BASE_DEBUG_ALIAS_H_ +#define PDFIUM_THIRD_PARTY_BASE_DEBUG_ALIAS_H_ +namespace pdfium { namespace base { namespace debug { @@ -28,5 +29,6 @@ void Alias(const void* var); } // namespace debug } // namespace base +} // namespace pdfium -#endif // BASE_DEBUG_ALIAS_H_ +#endif // PDFIUM_THIRD_PARTY_BASE_DEBUG_ALIAS_H_ -- cgit v1.2.3