Follow-up to #26: get rid of supplier_data, it's not really needed since

the caller can implement their own supplier object. r=mmentovai.


git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@27 4c0a9323-5329-0410-9bdc-e9ce6186880e

2 files changed, 3 insertions, 7 deletions

diff --git a/src/google/minidump_processor.h b/src/google/minidump_processor.h
index 9c80ee4e..31b80090 100644
--- a/src/google/minidump_processor.h
+++ b/src/google/minidump_processor.h
@@ -32,10 +32,8 @@ class MinidumpProcessor {
   ~MinidumpProcessor();
 
   // Fills in the given StackFrames vector by processing the minidump file.
-  // supplier_data is an opaque pointer which is passed to
-  // SymbolSupplier::GetSymbolFile().  Returns true on success.
-  bool Process(const string &minidump_file, void *supplier_data,
-               StackFrames *stack_frames);
+  // Returns true on success.
+  bool Process(const string &minidump_file, StackFrames *stack_frames);
 
  private:
   SymbolSupplier *supplier_;
diff --git a/src/google/symbol_supplier.h b/src/google/symbol_supplier.h
index f0a6e39c..74bddf7f 100644
--- a/src/google/symbol_supplier.h
+++ b/src/google/symbol_supplier.h
@@ -30,9 +30,7 @@ class SymbolSupplier {
   virtual ~SymbolSupplier() {}
 
   // Returns the path to the symbol file for the given module.
-  // supplier_data is passed through from MinidumpProcessor::Process().
-  virtual string GetSymbolFile(MinidumpModule *module,
-                               void *supplier_data) = 0;
+  virtual string GetSymbolFile(MinidumpModule *module) = 0;
 };
 
 }  // namespace google_airbag