diff options
author | jimblandy <jimblandy@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-02-09 17:13:17 +0000 |
---|---|---|
committer | jimblandy <jimblandy@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-02-09 17:13:17 +0000 |
commit | 952f3c7b7382b02e18c4bc70baa59751105786db (patch) | |
tree | 71868c73423058e73a93e30485cd26d1f292d659 /src/common/linux/dump_symbols.cc | |
parent | Breakpad Linux dumper: Add file comments as required by the style guide. (diff) | |
download | breakpad-952f3c7b7382b02e18c4bc70baa59751105786db.tar.xz |
Breakpad Linux dumper: Don't be silly about global functions.
Yes, classes are useful. But that doesn't mean that every function has
to gratuitously become a member function. The Google C++ Style Guide
does not require this silliness, since the function is in the
google_breakpad namespace anyway.
a=jimblandy, r=nealsid
git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@519 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/common/linux/dump_symbols.cc')
-rw-r--r-- | src/common/linux/dump_symbols.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/common/linux/dump_symbols.cc b/src/common/linux/dump_symbols.cc index 61dbee86..02d26987 100644 --- a/src/common/linux/dump_symbols.cc +++ b/src/common/linux/dump_symbols.cc @@ -355,8 +355,7 @@ std::string BaseFileName(const std::string &filename) { namespace google_breakpad { -bool DumpSymbols::WriteSymbolFile(const std::string &obj_file, - FILE *sym_file) { +bool WriteSymbolFile(const std::string &obj_file, FILE *sym_file) { int obj_fd = open(obj_file.c_str(), O_RDONLY); if (obj_fd < 0) { fprintf(stderr, "Failed to open ELF file '%s': %s\n", |