From 25b512d64dc9ec959d646151cf0cdc403ea0c335 Mon Sep 17 00:00:00 2001 From: jimblandy Date: Wed, 5 May 2010 17:35:10 +0000 Subject: Breakpad symbol dumper: Promise that Module::SetLoadAddress can be called at any time. It's possible to imagine an implementation of google_breakpad::Module in which calling SetLoadAddress at different times as the Module is populated would produce different output. For the Mac dumper, we'd like to depend on its current behavior --- that the load address is subtracted off only when writing the symbol file, and can be set at any time prior to that. This patch makes that promise part of Module's contract, and adjusts the test suite to verify that that promise is met. a=jimblandy, r=thestig git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@588 4c0a9323-5329-0410-9bdc-e9ce6186880e --- src/common/module.h | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'src/common/module.h') diff --git a/src/common/module.h b/src/common/module.h index 7a44d4ab..92012249 100644 --- a/src/common/module.h +++ b/src/common/module.h @@ -153,8 +153,14 @@ class Module { // for functions and lines will be written to the Breakpad symbol // file as offsets from this address. Construction initializes this // module's load address to zero: addresses written to the symbol - // file will be the same as they appear in the File and Line - // structures. + // file will be the same as they appear in the Function, Line, and + // StackFrameEntry structures. + // + // Note that this member function has no effect on addresses stored + // in the data added to this module; the Write member function + // simply subtracts off the load address from addresses before it + // prints them. Only the last load address given before calling + // Write is used. void SetLoadAddress(Address load_address); // Add FUNCTION to the module. @@ -223,7 +229,7 @@ class Module { // established by SetLoadAddress. bool Write(FILE *stream); -private: + private: // Report an error that has occurred writing the symbol file, using // errno to find the appropriate cause. Return false. -- cgit v1.2.1