jirentabu / crashrpt

Automatically exported from code.google.com/p/crashrpt
0 stars 0 forks source link

Avoid automatic install of crash filter #1

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
When using VC6 then the exception model requires one to use things like 
(Compiler option /EHa is on by default):

   _set_se_translator(CrashReport::SEH_Translator);
   ::SetUnhandledExceptionFilter(CrashReport::ReportCrash);

But when using VC7 / VC8 then one should use __try / __except (Compiler 
option /EHs is on by default).

Therefore it should be up to the application itself to decide how to 
install the handler. This would also make it easier for the application to 
decide what should happen in the handler (besides sending a crash report).

Also it would be nice if the interface for collecting the report items 
(screendump, minidump, xml-report,etc.), was seperated from building the 
report zip-file (AddFile, AddRegistry, etc.). This would allow the 
application it self to add own temporary report items and delete them 
after the crash-report was done.

Also it would be nice that the CrashRptDL.h was a wrapper class around the 
HMODULE handle and it had the necessary funktions to:
  - Create report items.
  - Build the report zip-file. 
  - Send / Save the report zip-file.

Will try to create an example of what I was thinking this coming weekend.

Original issue reported on code.google.com by swea...@gmail.com on 10 May 2007 at 9:08

GoogleCodeExporter commented 9 years ago
This may be already fixed in v1.1.

Original comment by zexspect...@gmail.com on 16 Jul 2009 at 2:53