From 5e62b34c0d6fa545b487b9b64fb4a04a0589bc13 Mon Sep 17 00:00:00 2001 From: Elliott Hughes Date: Thu, 25 Oct 2018 11:00:00 -0700 Subject: [PATCH] Clean up bionic_macros.h a bit. Use instead where possible, and move the bionic macros out of the way of the libbase ones. Yes, there are folks who manage to end up with both included at once (thanks OpenGL!), and cleaning that up doesn't seem nearly as practical as just making this change. Bug: N/A Test: builds Change-Id: I23fc544f39d5addf81dc61471771a5438778895b --- libc/bionic/grp_pwd_file.h | 6 ++-- libc/bionic/locale.cpp | 2 +- libc/bionic/malloc_info.cpp | 2 +- libc/bionic/pthread_atfork.cpp | 3 +- libc/bionic/pthread_internal.cpp | 2 +- libc/bionic/scandir.cpp | 2 +- libc/bionic/system_property_set.cpp | 2 +- libc/malloc_debug/DebugData.h | 2 +- libc/malloc_debug/GuardData.h | 6 ++-- libc/malloc_debug/MapData.h | 2 +- libc/malloc_debug/OptionData.h | 2 +- libc/malloc_debug/PointerData.h | 2 +- libc/malloc_debug/RecordData.h | 18 ++++++------ libc/malloc_debug/debug_disable.h | 2 +- libc/private/ErrnoRestorer.h | 7 ++--- libc/private/KernelArgumentBlock.h | 7 ++--- libc/private/ScopedPthreadMutexLocker.h | 7 ++--- libc/private/ScopedReaddir.h | 7 ++--- libc/private/ScopedSignalBlocker.h | 2 +- libc/private/WriteProtected.h | 9 ++---- libc/private/bionic_lock.h | 8 ++---- libc/private/bionic_macros.h | 28 ++++--------------- libc/private/bionic_systrace.h | 7 ++--- .../include/system_properties/context_node.h | 2 +- .../include/system_properties/prop_area.h | 4 +-- .../include/system_properties/prop_info.h | 2 +- .../system_properties/system_properties.h | 2 +- linker/Android.bp | 2 +- linker/linked_list.h | 2 +- linker/linker_block_allocator.h | 3 +- linker/linker_common_types.h | 2 ++ linker/linker_config.h | 3 +- linker/linker_logger.h | 4 ++- linker/linker_mapped_file_fragment.h | 2 +- tests/TemporaryFile.h | 2 +- tests/pthread_test.cpp | 2 +- 36 files changed, 68 insertions(+), 99 deletions(-) diff --git a/libc/bionic/grp_pwd_file.h b/libc/bionic/grp_pwd_file.h index 29d75f4d8..9004c4ef1 100644 --- a/libc/bionic/grp_pwd_file.h +++ b/libc/bionic/grp_pwd_file.h @@ -45,7 +45,7 @@ class MmapFile { bool FindByName(const char* name, Line* line); void Unmap(); - DISALLOW_COPY_AND_ASSIGN(MmapFile); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(MmapFile); private: enum class FileStatus { @@ -78,7 +78,7 @@ class PasswdFile { mmap_file_.Unmap(); } - DISALLOW_COPY_AND_ASSIGN(PasswdFile); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(PasswdFile); private: MmapFile mmap_file_; @@ -94,7 +94,7 @@ class GroupFile { mmap_file_.Unmap(); } - DISALLOW_COPY_AND_ASSIGN(GroupFile); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(GroupFile); private: MmapFile mmap_file_; diff --git a/libc/bionic/locale.cpp b/libc/bionic/locale.cpp index 2a5bcab9c..8358fb0ab 100644 --- a/libc/bionic/locale.cpp +++ b/libc/bionic/locale.cpp @@ -66,7 +66,7 @@ struct __locale_t { } } - DISALLOW_COPY_AND_ASSIGN(__locale_t); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(__locale_t); }; size_t __ctype_get_mb_cur_max() { diff --git a/libc/bionic/malloc_info.cpp b/libc/bionic/malloc_info.cpp index 99caedbad..9c8a4bf87 100644 --- a/libc/bionic/malloc_info.cpp +++ b/libc/bionic/malloc_info.cpp @@ -53,7 +53,7 @@ private: FILE* fp; const char* name; - DISALLOW_COPY_AND_ASSIGN(Elem); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(Elem); }; int malloc_info(int options, FILE* fp) { diff --git a/libc/bionic/pthread_atfork.cpp b/libc/bionic/pthread_atfork.cpp index 84e511c2e..fb12a3ba7 100644 --- a/libc/bionic/pthread_atfork.cpp +++ b/libc/bionic/pthread_atfork.cpp @@ -107,7 +107,7 @@ class atfork_list_t { atfork_t* first_; atfork_t* last_; - DISALLOW_COPY_AND_ASSIGN(atfork_list_t); + BIONIC_DISALLOW_COPY_AND_ASSIGN(atfork_list_t); }; static pthread_mutex_t g_atfork_list_mutex = PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP; @@ -180,4 +180,3 @@ extern "C" __LIBC_HIDDEN__ void __unregister_atfork(void* dso) { }); pthread_mutex_unlock(&g_atfork_list_mutex); } - diff --git a/libc/bionic/pthread_internal.cpp b/libc/bionic/pthread_internal.cpp index 829194cc7..92786fe7d 100644 --- a/libc/bionic/pthread_internal.cpp +++ b/libc/bionic/pthread_internal.cpp @@ -54,7 +54,7 @@ template class ScopedRWLock { private: pthread_rwlock_t* rwlock_; - DISALLOW_IMPLICIT_CONSTRUCTORS(ScopedRWLock); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(ScopedRWLock); }; typedef ScopedRWLock ScopedWriteLock; diff --git a/libc/bionic/scandir.cpp b/libc/bionic/scandir.cpp index e55be4209..0b390494d 100644 --- a/libc/bionic/scandir.cpp +++ b/libc/bionic/scandir.cpp @@ -90,7 +90,7 @@ class ScandirResult { return copy; } - DISALLOW_COPY_AND_ASSIGN(ScandirResult); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ScandirResult); }; int scandirat(int parent_fd, const char* dir_name, dirent*** name_list, diff --git a/libc/bionic/system_property_set.cpp b/libc/bionic/system_property_set.cpp index a70a37683..bc3ba7617 100644 --- a/libc/bionic/system_property_set.cpp +++ b/libc/bionic/system_property_set.cpp @@ -170,7 +170,7 @@ class SocketWriter { uint32_t uint_buf_[kUintBufSize]; size_t uint_buf_index_; - DISALLOW_IMPLICIT_CONSTRUCTORS(SocketWriter); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(SocketWriter); }; struct prop_msg { diff --git a/libc/malloc_debug/DebugData.h b/libc/malloc_debug/DebugData.h index f7cf8abd1..3a3629981 100644 --- a/libc/malloc_debug/DebugData.h +++ b/libc/malloc_debug/DebugData.h @@ -94,7 +94,7 @@ class DebugData { Config config_; - DISALLOW_COPY_AND_ASSIGN(DebugData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(DebugData); }; extern DebugData* g_debug; diff --git a/libc/malloc_debug/GuardData.h b/libc/malloc_debug/GuardData.h index 7b2167187..b6ec889c7 100644 --- a/libc/malloc_debug/GuardData.h +++ b/libc/malloc_debug/GuardData.h @@ -56,7 +56,7 @@ class GuardData : public OptionData { virtual const char* GetTypeName() = 0; - DISALLOW_COPY_AND_ASSIGN(GuardData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(GuardData); }; class FrontGuardData : public GuardData { @@ -75,7 +75,7 @@ class FrontGuardData : public GuardData { size_t offset_ = 0; - DISALLOW_COPY_AND_ASSIGN(FrontGuardData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(FrontGuardData); }; class RearGuardData : public GuardData { @@ -90,5 +90,5 @@ class RearGuardData : public GuardData { private: const char* GetTypeName() override { return "REAR"; } - DISALLOW_COPY_AND_ASSIGN(RearGuardData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(RearGuardData); }; diff --git a/libc/malloc_debug/MapData.h b/libc/malloc_debug/MapData.h index d8398bde3..b9b697cea 100644 --- a/libc/malloc_debug/MapData.h +++ b/libc/malloc_debug/MapData.h @@ -68,5 +68,5 @@ class MapData { std::mutex m_; std::set entries_; - DISALLOW_COPY_AND_ASSIGN(MapData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(MapData); }; diff --git a/libc/malloc_debug/OptionData.h b/libc/malloc_debug/OptionData.h index 3fa8e145d..8fb13a440 100644 --- a/libc/malloc_debug/OptionData.h +++ b/libc/malloc_debug/OptionData.h @@ -39,5 +39,5 @@ class OptionData { protected: DebugData* debug_; - DISALLOW_COPY_AND_ASSIGN(OptionData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(OptionData); }; diff --git a/libc/malloc_debug/PointerData.h b/libc/malloc_debug/PointerData.h index 62d4186f2..b05a76383 100644 --- a/libc/malloc_debug/PointerData.h +++ b/libc/malloc_debug/PointerData.h @@ -184,5 +184,5 @@ class PointerData : public OptionData { static std::mutex free_pointer_mutex_; static std::deque free_pointers_; - DISALLOW_COPY_AND_ASSIGN(PointerData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(PointerData); }; diff --git a/libc/malloc_debug/RecordData.h b/libc/malloc_debug/RecordData.h index 3e5ca029e..a01588214 100644 --- a/libc/malloc_debug/RecordData.h +++ b/libc/malloc_debug/RecordData.h @@ -49,7 +49,7 @@ class RecordEntry { pid_t tid_; private: - DISALLOW_COPY_AND_ASSIGN(RecordEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(RecordEntry); }; class ThreadCompleteEntry : public RecordEntry { @@ -60,7 +60,7 @@ class ThreadCompleteEntry : public RecordEntry { std::string GetString() const override; private: - DISALLOW_COPY_AND_ASSIGN(ThreadCompleteEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ThreadCompleteEntry); }; class AllocEntry : public RecordEntry { @@ -72,7 +72,7 @@ class AllocEntry : public RecordEntry { void* pointer_; private: - DISALLOW_COPY_AND_ASSIGN(AllocEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(AllocEntry); }; class MallocEntry : public AllocEntry { @@ -86,7 +86,7 @@ class MallocEntry : public AllocEntry { size_t size_; private: - DISALLOW_COPY_AND_ASSIGN(MallocEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(MallocEntry); }; class FreeEntry : public AllocEntry { @@ -97,7 +97,7 @@ class FreeEntry : public AllocEntry { std::string GetString() const override; private: - DISALLOW_COPY_AND_ASSIGN(FreeEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(FreeEntry); }; class CallocEntry : public MallocEntry { @@ -111,7 +111,7 @@ class CallocEntry : public MallocEntry { size_t nmemb_; private: - DISALLOW_COPY_AND_ASSIGN(CallocEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(CallocEntry); }; class ReallocEntry : public MallocEntry { @@ -125,7 +125,7 @@ class ReallocEntry : public MallocEntry { void* old_pointer_; private: - DISALLOW_COPY_AND_ASSIGN(ReallocEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ReallocEntry); }; // aligned_alloc, posix_memalign, memalign, pvalloc, valloc all recorded with this class. @@ -140,7 +140,7 @@ class MemalignEntry : public MallocEntry { size_t alignment_; private: - DISALLOW_COPY_AND_ASSIGN(MemalignEntry); + BIONIC_DISALLOW_COPY_AND_ASSIGN(MemalignEntry); }; class Config; @@ -170,5 +170,5 @@ class RecordData { std::atomic_bool dump_; std::string dump_file_; - DISALLOW_COPY_AND_ASSIGN(RecordData); + BIONIC_DISALLOW_COPY_AND_ASSIGN(RecordData); }; diff --git a/libc/malloc_debug/debug_disable.h b/libc/malloc_debug/debug_disable.h index 0049595e2..f9c3149ee 100644 --- a/libc/malloc_debug/debug_disable.h +++ b/libc/malloc_debug/debug_disable.h @@ -57,5 +57,5 @@ class ScopedDisableDebugCalls { private: bool disabled_; - DISALLOW_COPY_AND_ASSIGN(ScopedDisableDebugCalls); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ScopedDisableDebugCalls); }; diff --git a/libc/private/ErrnoRestorer.h b/libc/private/ErrnoRestorer.h index f4673936a..52e115a89 100644 --- a/libc/private/ErrnoRestorer.h +++ b/libc/private/ErrnoRestorer.h @@ -14,8 +14,7 @@ * limitations under the License. */ -#ifndef ERRNO_RESTORER_H -#define ERRNO_RESTORER_H +#pragma once #include @@ -37,7 +36,5 @@ class ErrnoRestorer { private: int saved_errno_; - DISALLOW_COPY_AND_ASSIGN(ErrnoRestorer); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ErrnoRestorer); }; - -#endif // ERRNO_RESTORER_H diff --git a/libc/private/KernelArgumentBlock.h b/libc/private/KernelArgumentBlock.h index e05ceb93e..886dd32a0 100644 --- a/libc/private/KernelArgumentBlock.h +++ b/libc/private/KernelArgumentBlock.h @@ -14,8 +14,7 @@ * limitations under the License. */ -#ifndef KERNEL_ARGUMENT_BLOCK_H -#define KERNEL_ARGUMENT_BLOCK_H +#pragma once #include #include @@ -71,7 +70,5 @@ class KernelArgumentBlock { libc_shared_globals* shared_globals; private: - DISALLOW_COPY_AND_ASSIGN(KernelArgumentBlock); + BIONIC_DISALLOW_COPY_AND_ASSIGN(KernelArgumentBlock); }; - -#endif // KERNEL_ARGUMENT_BLOCK_H diff --git a/libc/private/ScopedPthreadMutexLocker.h b/libc/private/ScopedPthreadMutexLocker.h index 58462e38b..1c1e4a730 100644 --- a/libc/private/ScopedPthreadMutexLocker.h +++ b/libc/private/ScopedPthreadMutexLocker.h @@ -14,8 +14,7 @@ * limitations under the License. */ -#ifndef SCOPED_PTHREAD_MUTEX_LOCKER_H -#define SCOPED_PTHREAD_MUTEX_LOCKER_H +#pragma once #include @@ -34,7 +33,5 @@ class ScopedPthreadMutexLocker { private: pthread_mutex_t* mu_; - DISALLOW_IMPLICIT_CONSTRUCTORS(ScopedPthreadMutexLocker); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(ScopedPthreadMutexLocker); }; - -#endif // SCOPED_PTHREAD_MUTEX_LOCKER_H diff --git a/libc/private/ScopedReaddir.h b/libc/private/ScopedReaddir.h index 1a59e1a21..dc22309ce 100644 --- a/libc/private/ScopedReaddir.h +++ b/libc/private/ScopedReaddir.h @@ -14,8 +14,7 @@ * limitations under the License. */ -#ifndef SCOPED_READDIR_H -#define SCOPED_READDIR_H +#pragma once #include @@ -47,7 +46,5 @@ class ScopedReaddir { private: DIR* dir_; - DISALLOW_COPY_AND_ASSIGN(ScopedReaddir); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ScopedReaddir); }; - -#endif // SCOPED_READDIR_H diff --git a/libc/private/ScopedSignalBlocker.h b/libc/private/ScopedSignalBlocker.h index d1cf629bf..10aacb368 100644 --- a/libc/private/ScopedSignalBlocker.h +++ b/libc/private/ScopedSignalBlocker.h @@ -46,5 +46,5 @@ class ScopedSignalBlocker { sigset64_t old_set_; - DISALLOW_COPY_AND_ASSIGN(ScopedSignalBlocker); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ScopedSignalBlocker); }; diff --git a/libc/private/WriteProtected.h b/libc/private/WriteProtected.h index 7a6b09800..69a68229b 100644 --- a/libc/private/WriteProtected.h +++ b/libc/private/WriteProtected.h @@ -14,8 +14,7 @@ * limitations under the License. */ -#ifndef _PRIVATE_WRITEPROTECTED_H -#define _PRIVATE_WRITEPROTECTED_H +#pragma once #include #include @@ -33,7 +32,7 @@ union WriteProtectedContents { char padding[PAGE_SIZE]; WriteProtectedContents() = default; - DISALLOW_COPY_AND_ASSIGN(WriteProtectedContents); + BIONIC_DISALLOW_COPY_AND_ASSIGN(WriteProtectedContents); } __attribute__((aligned(PAGE_SIZE))); // Write protected wrapper class that aligns its contents to a page boundary, @@ -49,7 +48,7 @@ class WriteProtected { public: WriteProtected() = default; - DISALLOW_COPY_AND_ASSIGN(WriteProtected); + BIONIC_DISALLOW_COPY_AND_ASSIGN(WriteProtected); void initialize() { // Not strictly necessary, but this will hopefully segfault if we initialize @@ -82,5 +81,3 @@ class WriteProtected { } } }; - -#endif diff --git a/libc/private/bionic_lock.h b/libc/private/bionic_lock.h index 54168d3ad..eebfeff54 100644 --- a/libc/private/bionic_lock.h +++ b/libc/private/bionic_lock.h @@ -25,8 +25,8 @@ * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. */ -#ifndef _BIONIC_LOCK_H -#define _BIONIC_LOCK_H + +#pragma once #include #include "private/bionic_futex.h" @@ -85,10 +85,8 @@ class LockGuard { lock_.unlock(); } - DISALLOW_COPY_AND_ASSIGN(LockGuard); + BIONIC_DISALLOW_COPY_AND_ASSIGN(LockGuard); private: Lock& lock_; }; - -#endif // _BIONIC_LOCK_H diff --git a/libc/private/bionic_macros.h b/libc/private/bionic_macros.h index 0a36cdb18..4800e3af9 100644 --- a/libc/private/bionic_macros.h +++ b/libc/private/bionic_macros.h @@ -14,31 +14,17 @@ * limitations under the License. */ -#ifndef _BIONIC_MACROS_H_ -#define _BIONIC_MACROS_H_ +#pragma once #include -// Frameworks OpenGL code currently leaks this header and allows -// collisions with other declarations, e.g., from libnativehelper. -// TODO: Remove once cleaned up. b/18334516 -#if !defined(DISALLOW_COPY_AND_ASSIGN) -// DISALLOW_COPY_AND_ASSIGN disallows the copy and operator= functions. -// It goes in the private: declarations in a class. -#define DISALLOW_COPY_AND_ASSIGN(TypeName) \ - TypeName(const TypeName&) = delete; \ +#define BIONIC_DISALLOW_COPY_AND_ASSIGN(TypeName) \ + TypeName(const TypeName&) = delete; \ void operator=(const TypeName&) = delete -#endif // !defined(DISALLOW_COPY_AND_ASSIGN) -// A macro to disallow all the implicit constructors, namely the -// default constructor, copy constructor and operator= functions. -// -// This should be used in the private: declarations for a class -// that wants to prevent anyone from instantiating it. This is -// especially useful for classes containing only static methods. -#define DISALLOW_IMPLICIT_CONSTRUCTORS(TypeName) \ - TypeName() = delete; \ - DISALLOW_COPY_AND_ASSIGN(TypeName) +#define BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(TypeName) \ + TypeName() = delete; \ + BIONIC_DISALLOW_COPY_AND_ASSIGN(TypeName) #define BIONIC_ROUND_UP_POWER_OF_2(value) \ ((sizeof(value) == 8) \ @@ -101,5 +87,3 @@ char (&ArraySizeHelper(T (&array)[N]))[N]; // NOLINT(readability/casting) #else #define __BIONIC_FALLTHROUGH #endif - -#endif // _BIONIC_MACROS_H_ diff --git a/libc/private/bionic_systrace.h b/libc/private/bionic_systrace.h index 304fb8061..86d2a08c5 100644 --- a/libc/private/bionic_systrace.h +++ b/libc/private/bionic_systrace.h @@ -14,8 +14,7 @@ * limitations under the License. */ -#ifndef BIONIC_SYSTRACE_H -#define BIONIC_SYSTRACE_H +#pragma once #include "bionic_macros.h" @@ -31,10 +30,8 @@ class __LIBC_HIDDEN__ ScopedTrace { void End(); private: bool called_end_; - DISALLOW_COPY_AND_ASSIGN(ScopedTrace); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ScopedTrace); }; void bionic_trace_begin(const char* message); void bionic_trace_end(); - -#endif diff --git a/libc/system_properties/include/system_properties/context_node.h b/libc/system_properties/include/system_properties/context_node.h index 35d0e92c2..20f40133f 100644 --- a/libc/system_properties/include/system_properties/context_node.h +++ b/libc/system_properties/include/system_properties/context_node.h @@ -42,7 +42,7 @@ class ContextNode { Unmap(); } - DISALLOW_COPY_AND_ASSIGN(ContextNode); + BIONIC_DISALLOW_COPY_AND_ASSIGN(ContextNode); bool Open(bool access_rw, bool* fsetxattr_failed); bool CheckAccessAndOpen(); diff --git a/libc/system_properties/include/system_properties/prop_area.h b/libc/system_properties/include/system_properties/prop_area.h index 2c253370e..a69f90eb1 100644 --- a/libc/system_properties/include/system_properties/prop_area.h +++ b/libc/system_properties/include/system_properties/prop_area.h @@ -86,7 +86,7 @@ struct prop_bt { } private: - DISALLOW_COPY_AND_ASSIGN(prop_bt); + BIONIC_DISALLOW_COPY_AND_ASSIGN(prop_bt); }; class prop_area { @@ -158,5 +158,5 @@ class prop_area { uint32_t reserved_[28]; char data_[0]; - DISALLOW_COPY_AND_ASSIGN(prop_area); + BIONIC_DISALLOW_COPY_AND_ASSIGN(prop_area); }; diff --git a/libc/system_properties/include/system_properties/prop_info.h b/libc/system_properties/include/system_properties/prop_info.h index a127550c2..27b29c8b7 100644 --- a/libc/system_properties/include/system_properties/prop_info.h +++ b/libc/system_properties/include/system_properties/prop_info.h @@ -83,7 +83,7 @@ struct prop_info { prop_info(const char* name, uint32_t namelen, uint32_t long_offset); private: - DISALLOW_IMPLICIT_CONSTRUCTORS(prop_info); + BIONIC_DISALLOW_IMPLICIT_CONSTRUCTORS(prop_info); }; static_assert(sizeof(prop_info) == 96, "sizeof struct prop_info must be 96 bytes"); diff --git a/libc/system_properties/include/system_properties/system_properties.h b/libc/system_properties/include/system_properties/system_properties.h index 52ffcafd0..cad29cc72 100644 --- a/libc/system_properties/include/system_properties/system_properties.h +++ b/libc/system_properties/include/system_properties/system_properties.h @@ -52,7 +52,7 @@ class SystemProperties { explicit SystemProperties(bool initialized) : initialized_(initialized) { } - DISALLOW_COPY_AND_ASSIGN(SystemProperties); + BIONIC_DISALLOW_COPY_AND_ASSIGN(SystemProperties); bool Init(const char* filename); bool AreaInit(const char* filename, bool* fsetxattr_failed); diff --git a/linker/Android.bp b/linker/Android.bp index 697c26089..779cd3f99 100644 --- a/linker/Android.bp +++ b/linker/Android.bp @@ -15,7 +15,7 @@ cc_library_static { // We need to access Bionic private headers in the linker. include_dirs: ["bionic/libc"], - static_libs: ["libasync_safe"], + static_libs: ["libasync_safe", "libbase"], } // This is used for bionic on (host) Linux to bootstrap our linker embedded into diff --git a/linker/linked_list.h b/linker/linked_list.h index 7f70a2c7b..5473ca0d6 100644 --- a/linker/linked_list.h +++ b/linker/linked_list.h @@ -28,7 +28,7 @@ #pragma once -#include "private/bionic_macros.h" +#include template struct LinkedListEntry { diff --git a/linker/linker_block_allocator.h b/linker/linker_block_allocator.h index b501659fe..bd44fc829 100644 --- a/linker/linker_block_allocator.h +++ b/linker/linker_block_allocator.h @@ -30,7 +30,8 @@ #include #include -#include "private/bionic_macros.h" + +#include struct LinkerBlockAllocatorPage; diff --git a/linker/linker_common_types.h b/linker/linker_common_types.h index ffa406689..ae78aa99b 100644 --- a/linker/linker_common_types.h +++ b/linker/linker_common_types.h @@ -31,6 +31,8 @@ #include #include "linked_list.h" +#include + // TODO(dimitry): move this to linker_defines.h? Unless it is removed by // consequent refactoring steps. diff --git a/linker/linker_config.h b/linker/linker_config.h index e117aeaad..24c44f4f3 100644 --- a/linker/linker_config.h +++ b/linker/linker_config.h @@ -32,13 +32,14 @@ #include #include -#include "private/bionic_macros.h" #include #include #include #include +#include + class NamespaceLinkConfig { public: NamespaceLinkConfig() = default; diff --git a/linker/linker_logger.h b/linker/linker_logger.h index 9ce438eae..18287996e 100644 --- a/linker/linker_logger.h +++ b/linker/linker_logger.h @@ -30,9 +30,11 @@ #include #include -#include "private/bionic_macros.h" + #include "private/bionic_systrace.h" +#include + #define LD_LOG(type, x...) \ { \ g_linker_logger.Log(type, x); \ diff --git a/linker/linker_mapped_file_fragment.h b/linker/linker_mapped_file_fragment.h index f7872bdb3..91e094fd4 100644 --- a/linker/linker_mapped_file_fragment.h +++ b/linker/linker_mapped_file_fragment.h @@ -30,7 +30,7 @@ #include -#include "private/bionic_macros.h" +#include class MappedFileFragment { public: diff --git a/tests/TemporaryFile.h b/tests/TemporaryFile.h index 8af92d4d7..7853781a4 100644 --- a/tests/TemporaryFile.h +++ b/tests/TemporaryFile.h @@ -17,7 +17,7 @@ #include #include -#include "private/bionic_macros.h" +#include template class GenericTemporaryFile { diff --git a/tests/pthread_test.cpp b/tests/pthread_test.cpp index e68f1ff03..84ce531f0 100644 --- a/tests/pthread_test.cpp +++ b/tests/pthread_test.cpp @@ -34,12 +34,12 @@ #include #include +#include #include #include #include #include "private/bionic_constants.h" -#include "private/bionic_macros.h" #include "BionicDeathTest.h" #include "SignalUtils.h" #include "utils.h"