diff options
author | Ted Mielczarek <ted@mielczarek.org> | 2016-06-10 13:23:29 -0400 |
---|---|---|
committer | Ted Mielczarek <ted@mielczarek.org> | 2016-06-10 13:23:29 -0400 |
commit | c44217f6468152bf5693df7ec78a48d97e2b0e83 (patch) | |
tree | f05595f7a39b34c8111c6a74bfbf505c66dfa4b1 /src/common/module.h | |
parent | Fix a trivial parsing bug caught by static analysis (diff) | |
download | breakpad-c44217f6468152bf5693df7ec78a48d97e2b0e83.tar.xz |
Dump INFO CODE_ID containing Build ID in Linux dump_syms
I'd like to have the Build ID available for our symbol server
uploading, and this will make it easy.
Most of this change is me rewriting dump_symbols_unittest to be
typed tests so I could add a new test there.
R=mark@chromium.org
BUG=
Review URL: https://codereview.chromium.org/2052263002 .
Diffstat (limited to 'src/common/module.h')
-rw-r--r-- | src/common/module.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/common/module.h b/src/common/module.h index 65b5595d..6c2bb278 100644 --- a/src/common/module.h +++ b/src/common/module.h @@ -179,7 +179,7 @@ class Module { // Create a new module with the given name, operating system, // architecture, and ID string. Module(const string &name, const string &os, const string &architecture, - const string &id); + const string &id, const string &code_id = ""); ~Module(); // Set the module's load address to LOAD_ADDRESS; addresses given @@ -281,6 +281,7 @@ class Module { string os() const { return os_; } string architecture() const { return architecture_; } string identifier() const { return id_; } + string code_identifier() const { return code_id_; } private: // Report an error that has occurred writing the symbol file, using @@ -293,7 +294,7 @@ class Module { static bool WriteRuleMap(const RuleMap &rule_map, std::ostream &stream); // Module header entries. - string name_, os_, architecture_, id_; + string name_, os_, architecture_, id_, code_id_; // The module's nominal load address. Addresses for functions and // lines are absolute, assuming the module is loaded at this |