X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/61ad154c0146285f027db943e6aae33f20310a20..694f70fa9c0cc8889c208020bf75b2e3861f6818:/src/common/debugrpt.cpp diff --git a/src/common/debugrpt.cpp b/src/common/debugrpt.cpp index ef86029ab8..2f7a5e602b 100644 --- a/src/common/debugrpt.cpp +++ b/src/common/debugrpt.cpp @@ -27,6 +27,7 @@ #include "wx/app.h" #include "wx/log.h" #include "wx/intl.h" + #include "wx/utils.h" #endif // WX_PRECOMP #if wxUSE_DEBUGREPORT @@ -52,12 +53,14 @@ #include "wx/zipstrm.h" #endif // wxUSE_ZIPSTREAM -#if wxUSE_STACKWALKER +WX_CHECK_BUILD_OPTIONS("wxQA") // ---------------------------------------------------------------------------- // XmlStackWalker: stack walker specialization which dumps stack in XML // ---------------------------------------------------------------------------- +#if wxUSE_STACKWALKER + class XmlStackWalker : public wxStackWalker { public: @@ -178,7 +181,7 @@ void XmlStackWalker::OnStackFrame(const wxStackFrame& frame) wxDebugReport::wxDebugReport() { // get a temporary directory name - wxString appname(wxTheApp ? wxTheApp->GetAppName() : _T("wx")); + wxString appname = GetReportName(); // we can't use CreateTempFileName() because it creates a file, not a // directory, so do our best to create a unique name ourselves @@ -223,7 +226,12 @@ wxDebugReport::~wxDebugReport() if ( !m_dir.empty() ) { - if ( wxRmDir(m_dir) != 0 ) + // Temp fix: what should this be? eVC++ doesn't like wxRmDir +#ifdef __WXWINCE__ + if ( wxRmdir(m_dir.fn_str()) != 0 ) +#else + if ( wxRmDir(m_dir.fn_str()) != 0 ) +#endif { wxLogSysError(_("Failed to clean up debug report directory \"%s\""), m_dir.c_str()); @@ -237,7 +245,10 @@ wxDebugReport::~wxDebugReport() wxString wxDebugReport::GetReportName() const { - return wxString(wxTheApp ? wxTheApp->GetAppName() : _T("wx")); + if(wxTheApp) + return wxTheApp->GetAppName(); + + return _T("wx"); } void wxDebugReport::AddFile(const wxString& name, const wxString& description) @@ -279,6 +290,10 @@ void wxDebugReport::AddAll(Context context) #if wxUSE_CRASHREPORT AddDump(context); #endif // wxUSE_CRASHREPORT + +#if !wxUSE_STACKWALKER && !wxUSE_CRASHREPORT + wxUnusedVar(context); +#endif } // ---------------------------------------------------------------------------- @@ -387,8 +402,8 @@ bool wxDebugReport::AddContext(wxDebugReport::Context ctx) wxXmlNode *nodeRoot = new wxXmlNode(wxXML_ELEMENT_NODE, _T("report")); xmldoc.SetRoot(nodeRoot); nodeRoot->AddProperty(_T("version"), _T("1.0")); - nodeRoot->AddProperty(_T("kind"), ctx == Context_Curent ? _T("user") - : _T("exception")); + nodeRoot->AddProperty(_T("kind"), ctx == Context_Current ? _T("user") + : _T("exception")); // add system information wxXmlNode *nodeSystemInfo = new wxXmlNode(wxXML_ELEMENT_NODE, _T("system")); @@ -423,7 +438,7 @@ bool wxDebugReport::AddContext(wxDebugReport::Context ctx) { sw.WalkFromException(); } - else // Context_Curent + else // Context_Current { sw.Walk(); } @@ -560,7 +575,8 @@ bool wxDebugReportCompress::DoProcess() if ( !zos.PutNextEntry(ze) ) return false; - wxFFileInputStream is(wxFileName(fn.GetPath(), name).GetFullPath()); + wxFileName filename(fn.GetPath(), name); + wxFFileInputStream is(filename.GetFullPath()); if ( !is.IsOk() || !zos.Write(is).IsOk() ) return false; }