From e1930985430ce289f4fe8525f51050e5d78cc44e Mon Sep 17 00:00:00 2001 From: "ted.mielczarek" Date: Fri, 25 Jun 2010 16:57:07 +0000 Subject: Breakpad: Avoid using the C++ headers. This patch avoids unnecessary use of the headers in files that don't actually use the identifiers they declare in the std:: namespace. It also changes some files to better conform with the "Names and Order of Includes" rules in the Google C++ Style Guide. A=jimb R=mark git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@619 4c0a9323-5329-0410-9bdc-e9ce6186880e --- src/common/dwarf/bytereader-inl.h | 4 ++-- src/common/dwarf/cfi_assembler.cc | 6 +++--- src/common/dwarf/dwarf2diehandler.cc | 4 ++-- src/common/dwarf/dwarf2reader.cc | 10 ++++++---- src/common/dwarf/dwarf2reader_cfi_unittest.cc | 9 +++++---- 5 files changed, 18 insertions(+), 15 deletions(-) (limited to 'src/common/dwarf') diff --git a/src/common/dwarf/bytereader-inl.h b/src/common/dwarf/bytereader-inl.h index 57fce448..3c167089 100644 --- a/src/common/dwarf/bytereader-inl.h +++ b/src/common/dwarf/bytereader-inl.h @@ -29,10 +29,10 @@ #ifndef UTIL_DEBUGINFO_BYTEREADER_INL_H__ #define UTIL_DEBUGINFO_BYTEREADER_INL_H__ -#include - #include "common/dwarf/bytereader.h" +#include + namespace dwarf2reader { inline uint8 ByteReader::ReadOneByte(const char* buffer) const { diff --git a/src/common/dwarf/cfi_assembler.cc b/src/common/dwarf/cfi_assembler.cc index 50e92e5a..a6a5aca6 100644 --- a/src/common/dwarf/cfi_assembler.cc +++ b/src/common/dwarf/cfi_assembler.cc @@ -32,11 +32,11 @@ // cfi_assembler.cc: Implementation of google_breakpad::CFISection class. // See cfi_assembler.h for details. -#include -#include - #include "common/dwarf/cfi_assembler.h" +#include +#include + namespace google_breakpad { using dwarf2reader::DwarfPointerEncoding; diff --git a/src/common/dwarf/dwarf2diehandler.cc b/src/common/dwarf/dwarf2diehandler.cc index 099f5193..5d019293 100644 --- a/src/common/dwarf/dwarf2diehandler.cc +++ b/src/common/dwarf/dwarf2diehandler.cc @@ -31,10 +31,10 @@ // dwarf2diehandler.cc: Implement the dwarf2reader::DieDispatcher class. // See dwarf2diehandler.h for details. -#include - #include "common/dwarf/dwarf2diehandler.h" +#include + namespace dwarf2reader { DIEDispatcher::~DIEDispatcher() { diff --git a/src/common/dwarf/dwarf2reader.cc b/src/common/dwarf/dwarf2reader.cc index 1a397019..a2915bad 100644 --- a/src/common/dwarf/dwarf2reader.cc +++ b/src/common/dwarf/dwarf2reader.cc @@ -31,16 +31,18 @@ // Implementation of dwarf2reader::LineInfo, dwarf2reader::CompilationUnit, // and dwarf2reader::CallFrameInfo. See dwarf2reader.h for details. -#include -#include -#include +#include "common/dwarf/dwarf2reader.h" + +#include +#include +#include + #include #include #include #include #include "common/dwarf/bytereader-inl.h" -#include "common/dwarf/dwarf2reader.h" #include "common/dwarf/bytereader.h" #include "common/dwarf/line_state_machine.h" diff --git a/src/common/dwarf/dwarf2reader_cfi_unittest.cc b/src/common/dwarf/dwarf2reader_cfi_unittest.cc index 85889f88..1a2431c7 100644 --- a/src/common/dwarf/dwarf2reader_cfi_unittest.cc +++ b/src/common/dwarf/dwarf2reader_cfi_unittest.cc @@ -31,7 +31,8 @@ // dwarf2reader_cfi_unittest.cc: Unit tests for dwarf2reader::CallFrameInfo -#include +#include + #include // The '.eh_frame' format, used by the Linux C++ ABI for exception @@ -45,9 +46,9 @@ // interpretation against the test's intentions. Each ELF file is named // "cfitest-TEST", where TEST identifies the particular test. #ifdef WRITE_ELF -#include -#include -#include +#include +#include +#include extern "C" { // To compile with WRITE_ELF, you should add the 'include' directory // of the binutils, gcc, or gdb source tree to your #include path; -- cgit v1.2.1