From 09b056975dacd1f0f815ad820b6dc9913b0118a3 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Tue, 23 Jun 2020 18:55:43 -0400 Subject: fix pointer style to match the style guide We do this in a lot of places, but we're inconsistent. Normalize the code to the Google C++ style guide. Change-Id: Ic2aceab661ce8f6b993dda21b1cdf5d2198dcbbf Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2262932 Reviewed-by: Sterling Augustine Reviewed-by: Mark Mentovai --- src/processor/simple_serializer-inl.h | 44 +++++++++++++++++------------------ 1 file changed, 22 insertions(+), 22 deletions(-) (limited to 'src/processor/simple_serializer-inl.h') diff --git a/src/processor/simple_serializer-inl.h b/src/processor/simple_serializer-inl.h index 606bb3ce..cb1a0408 100644 --- a/src/processor/simple_serializer-inl.h +++ b/src/processor/simple_serializer-inl.h @@ -56,12 +56,12 @@ class SimpleSerializer { public: static size_t SizeOf(bool boolean) { return 1; } - static char *Write(bool boolean, char *dest) { + static char* Write(bool boolean, char* dest) { *dest = static_cast(boolean? 255 : 0); return ++dest; } - static const char *Read(const char *source, bool *value) { + static const char* Read(const char* source, bool* value) { *value = ((*source) == 0 ? false : true); return ++source; } @@ -71,9 +71,9 @@ class SimpleSerializer { template<> class SimpleSerializer { public: - static size_t SizeOf(const string &str) { return str.size() + 1; } + static size_t SizeOf(const string& str) { return str.size() + 1; } - static char *Write(const string &str, char *dest) { + static char* Write(const string& str, char* dest) { strcpy(dest, str.c_str()); return dest + SizeOf(str); } @@ -83,11 +83,11 @@ class SimpleSerializer { template<> class SimpleSerializer { public: - static size_t SizeOf(const char *cstring) { + static size_t SizeOf(const char* cstring) { return strlen(cstring) + 1; } - static char *Write(const char *cstring, char *dest) { + static char* Write(const char* cstring, char* dest) { strcpy(dest, cstring); return dest + SizeOf(cstring); } @@ -98,13 +98,13 @@ template<> class SimpleSerializer { typedef BasicSourceLineResolver::Line Line; public: - static size_t SizeOf(const Line &line) { + static size_t SizeOf(const Line& line) { return SimpleSerializer::SizeOf(line.address) + SimpleSerializer::SizeOf(line.size) + SimpleSerializer::SizeOf(line.source_file_id) + SimpleSerializer::SizeOf(line.line); } - static char *Write(const Line &line, char *dest) { + static char* Write(const Line& line, char* dest) { dest = SimpleSerializer::Write(line.address, dest); dest = SimpleSerializer::Write(line.size, dest); dest = SimpleSerializer::Write(line.source_file_id, dest); @@ -118,12 +118,12 @@ template<> class SimpleSerializer { typedef BasicSourceLineResolver::PublicSymbol PublicSymbol; public: - static size_t SizeOf(const PublicSymbol &pubsymbol) { + static size_t SizeOf(const PublicSymbol& pubsymbol) { return SimpleSerializer::SizeOf(pubsymbol.name) + SimpleSerializer::SizeOf(pubsymbol.address) + SimpleSerializer::SizeOf(pubsymbol.parameter_size); } - static char *Write(const PublicSymbol &pubsymbol, char *dest) { + static char* Write(const PublicSymbol& pubsymbol, char* dest) { dest = SimpleSerializer::Write(pubsymbol.name, dest); dest = SimpleSerializer::Write(pubsymbol.address, dest); dest = SimpleSerializer::Write(pubsymbol.parameter_size, dest); @@ -135,7 +135,7 @@ class SimpleSerializer { template<> class SimpleSerializer { public: - static size_t SizeOf(const WindowsFrameInfo &wfi) { + static size_t SizeOf(const WindowsFrameInfo& wfi) { unsigned int size = 0; size += sizeof(int32_t); // wfi.type_ size += SimpleSerializer::SizeOf(wfi.valid); @@ -149,7 +149,7 @@ class SimpleSerializer { size += SimpleSerializer::SizeOf(wfi.program_string); return size; } - static char *Write(const WindowsFrameInfo &wfi, char *dest) { + static char* Write(const WindowsFrameInfo& wfi, char* dest) { dest = SimpleSerializer::Write( static_cast(wfi.type_), dest); dest = SimpleSerializer::Write(wfi.valid, dest); @@ -170,11 +170,11 @@ template<> class SimpleSerializer< linked_ptr > { typedef BasicSourceLineResolver::Line Line; public: - static size_t SizeOf(const linked_ptr &lineptr) { + static size_t SizeOf(const linked_ptr& lineptr) { if (lineptr.get() == NULL) return 0; return SimpleSerializer::SizeOf(*(lineptr.get())); } - static char *Write(const linked_ptr &lineptr, char *dest) { + static char* Write(const linked_ptr& lineptr, char* dest) { if (lineptr.get()) dest = SimpleSerializer::Write(*(lineptr.get()), dest); return dest; @@ -187,7 +187,7 @@ class SimpleSerializer { typedef BasicSourceLineResolver::Function Function; typedef BasicSourceLineResolver::Line Line; public: - static size_t SizeOf(const Function &func) { + static size_t SizeOf(const Function& func) { unsigned int size = 0; size += SimpleSerializer::SizeOf(func.name); size += SimpleSerializer::SizeOf(func.address); @@ -197,7 +197,7 @@ class SimpleSerializer { return size; } - static char *Write(const Function &func, char *dest) { + static char* Write(const Function& func, char* dest) { dest = SimpleSerializer::Write(func.name, dest); dest = SimpleSerializer::Write(func.address, dest); dest = SimpleSerializer::Write(func.size, dest); @@ -214,12 +214,12 @@ template<> class SimpleSerializer< linked_ptr > { typedef BasicSourceLineResolver::Function Function; public: - static size_t SizeOf(const linked_ptr &func) { + static size_t SizeOf(const linked_ptr& func) { if (!func.get()) return 0; return SimpleSerializer::SizeOf(*(func.get())); } - static char *Write(const linked_ptr &func, char *dest) { + static char* Write(const linked_ptr& func, char* dest) { if (func.get()) dest = SimpleSerializer::Write(*(func.get()), dest); return dest; @@ -230,11 +230,11 @@ template<> class SimpleSerializer< linked_ptr > { typedef BasicSourceLineResolver::PublicSymbol PublicSymbol; public: - static size_t SizeOf(const linked_ptr &pubsymbol) { + static size_t SizeOf(const linked_ptr& pubsymbol) { if (pubsymbol.get() == NULL) return 0; return SimpleSerializer::SizeOf(*(pubsymbol.get())); } - static char *Write(const linked_ptr &pubsymbol, char *dest) { + static char* Write(const linked_ptr& pubsymbol, char* dest) { if (pubsymbol.get()) dest = SimpleSerializer::Write(*(pubsymbol.get()), dest); return dest; @@ -244,11 +244,11 @@ class SimpleSerializer< linked_ptr > { template<> class SimpleSerializer< linked_ptr > { public: - static size_t SizeOf(const linked_ptr &wfi) { + static size_t SizeOf(const linked_ptr& wfi) { if (wfi.get() == NULL) return 0; return SimpleSerializer::SizeOf(*(wfi.get())); } - static char *Write(const linked_ptr &wfi, char *dest) { + static char* Write(const linked_ptr& wfi, char* dest) { if (wfi.get()) dest = SimpleSerializer::Write(*(wfi.get()), dest); return dest; -- cgit v1.2.1