From 257123ca70f1895cd6fd37c75ebaeef2de9e17b1 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Tue, 29 Dec 2015 13:42:49 -0800 Subject: Let breakpad build with -Wall on OS X and Linux. A=thakis@chromium.org Original Review: https://codereview.chromium.org/1550933002/ R=thakis@chromium.org Review URL: https://codereview.chromium.org/1554613002 . --- src/common/linux/dump_symbols.cc | 1 - src/common/linux/libcurl_wrapper.h | 2 +- src/common/linux/synth_elf.cc | 4 +++- src/common/linux/synth_elf.h | 2 ++ 4 files changed, 6 insertions(+), 3 deletions(-) (limited to 'src/common/linux') diff --git a/src/common/linux/dump_symbols.cc b/src/common/linux/dump_symbols.cc index 1e96ca6a..5ae44e53 100644 --- a/src/common/linux/dump_symbols.cc +++ b/src/common/linux/dump_symbols.cc @@ -889,7 +889,6 @@ bool ReadSymbolDataElfClass(const typename ElfClass::Ehdr* elf_header, const DumpOptions& options, Module** out_module) { typedef typename ElfClass::Ehdr Ehdr; - typedef typename ElfClass::Shdr Shdr; *out_module = NULL; diff --git a/src/common/linux/libcurl_wrapper.h b/src/common/linux/libcurl_wrapper.h index de84a63b..25905ad8 100644 --- a/src/common/linux/libcurl_wrapper.h +++ b/src/common/linux/libcurl_wrapper.h @@ -43,7 +43,7 @@ namespace google_breakpad { class LibcurlWrapper { public: LibcurlWrapper(); - ~LibcurlWrapper(); + virtual ~LibcurlWrapper(); virtual bool Init(); virtual bool SetProxy(const string& proxy_host, const string& proxy_userpwd); diff --git a/src/common/linux/synth_elf.cc b/src/common/linux/synth_elf.cc index b978550f..98e81dab 100644 --- a/src/common/linux/synth_elf.cc +++ b/src/common/linux/synth_elf.cc @@ -213,8 +213,10 @@ void ELF::Finish() { SymbolTable::SymbolTable(Endianness endianness, size_t addr_size, StringTable& table) : Section(endianness), - addr_size_(addr_size), table_(table) { +#ifndef NDEBUG + addr_size_ = addr_size; +#endif assert(addr_size_ == 4 || addr_size_ == 8); } diff --git a/src/common/linux/synth_elf.h b/src/common/linux/synth_elf.h index 330ceae8..1d2a20ca 100644 --- a/src/common/linux/synth_elf.h +++ b/src/common/linux/synth_elf.h @@ -173,7 +173,9 @@ class SymbolTable : public Section { uint64_t size, unsigned info, uint16_t shndx); private: +#ifndef NDEBUG size_t addr_size_; +#endif StringTable& table_; }; -- cgit v1.2.1