From 2535b1afcf6139c2daf16b9bdb4af38dd71279e1 Mon Sep 17 00:00:00 2001 From: glx Date: Thu, 19 Nov 2009 16:27:15 +0000 Subject: (svn r18184) -Codechange: add crash screenshot for win32 --- src/os/windows/crashlog_win.cpp | 26 ++++++++++++++++++++------ src/os/windows/ottdres.rc.in | 12 ++++++------ 2 files changed, 26 insertions(+), 12 deletions(-) diff --git a/src/os/windows/crashlog_win.cpp b/src/os/windows/crashlog_win.cpp index 11acad0ab..cf17615c9 100644 --- a/src/os/windows/crashlog_win.cpp +++ b/src/os/windows/crashlog_win.cpp @@ -44,6 +44,8 @@ public: char crashlog_filename[MAX_PATH]; /** Buffer for the filename of the crash dump */ char crashdump_filename[MAX_PATH]; + /** Buffer for the filename of the crash screenshot */ + char screenshot_filename[MAX_PATH]; /** * A crash log is always generated when it's generated. @@ -55,6 +57,7 @@ public: this->crashlog[0] = '\0'; this->crashlog_filename[0] = '\0'; this->crashdump_filename[0] = '\0'; + this->screenshot_filename[0] = '\0'; } /** @@ -380,6 +383,7 @@ static LONG WINAPI ExceptionHandler(EXCEPTION_POINTERS *ep) log->FillCrashLog(log->crashlog, lastof(log->crashlog)); log->WriteCrashLog(log->crashlog, log->crashlog_filename, lastof(log->crashlog_filename)); log->WriteCrashDump(log->crashdump_filename, lastof(log->crashdump_filename)); + log->WriteScreenshot(log->screenshot_filename, lastof(log->screenshot_filename)); /* Close any possible log files */ CloseConsoleLogIfActive(); @@ -430,10 +434,10 @@ static const TCHAR _crash_desc[] = _T("This will greatly help debugging. The correct place to do this is http://bugs.openttd.org. ") _T("The information contained in the report is displayed below.\n") _T("Press \"Emergency save\" to attempt saving the game. Generated file(s):\n") - _T("'%s%s%s'"); + _T("%s"); static const TCHAR _save_succeeded[] = - _T("Emergency save succeeded. Its location is '%s'.\n") + _T("Emergency save succeeded.\nIts location is '%s'.\n") _T("Be aware that critical parts of the internal game state may have become ") _T("corrupted. The saved game is not guaranteed to work."); @@ -481,11 +485,21 @@ static INT_PTR CALLBACK CrashDialogFunc(HWND wnd, UINT msg, WPARAM wParam, LPARA *p = '\0'; /* Add path to crash.log and crash.dmp (if any) to the crash window text */ - size_t len = _tcslen(_crash_desc) + _tcslen(OTTD2FS(CrashLogWindows::current->crashlog_filename)) + _tcslen(OTTD2FS(CrashLogWindows::current->crashdump_filename)) + 4; + size_t len = _tcslen(_crash_desc) + 2; + len += _tcslen(OTTD2FS(CrashLogWindows::current->crashlog_filename)) + 2; + len += _tcslen(OTTD2FS(CrashLogWindows::current->crashdump_filename)) + 2; + len += _tcslen(OTTD2FS(CrashLogWindows::current->screenshot_filename)) + 1; + TCHAR *text = AllocaM(TCHAR, len); - TCHAR *dump = _tcsdup(OTTD2FS(CrashLogWindows::current->crashdump_filename)); - _sntprintf(text, len, _crash_desc, OTTD2FS(CrashLogWindows::current->crashlog_filename), dump[0] != _T('\0') ? _T("'\n'") : _T(""), dump); - free(dump); + _sntprintf(text, len, _crash_desc, OTTD2FS(CrashLogWindows::current->crashlog_filename)); + if (OTTD2FS(CrashLogWindows::current->crashdump_filename)[0] != _T('\0')) { + _tcscat(text, _T("\n")); + _tcscat(text, OTTD2FS(CrashLogWindows::current->crashdump_filename)); + } + if (OTTD2FS(CrashLogWindows::current->screenshot_filename)[0] != _T('\0')) { + _tcscat(text, _T("\n")); + _tcscat(text, OTTD2FS(CrashLogWindows::current->screenshot_filename)); + } SetDlgItemText(wnd, 10, text); SetDlgItemText(wnd, 11, MB_TO_WIDE_BUFFER(dos_nl, crash_msgW, lengthof(crash_msgW))); diff --git a/src/os/windows/ottdres.rc.in b/src/os/windows/ottdres.rc.in index 451547bd1..b91ee4178 100644 --- a/src/os/windows/ottdres.rc.in +++ b/src/os/windows/ottdres.rc.in @@ -46,17 +46,17 @@ LANGUAGE LANG_NEUTRAL, SUBLANG_DEFAULT // Dialog // -100 DIALOG DISCARDABLE 0, 0, 305, 91 +100 DIALOG DISCARDABLE 0, 0, 305, 99 STYLE DS_MODALFRAME | WS_POPUP | WS_CAPTION | WS_SYSMENU CAPTION "Fatal Application Failure" FONT 8, "MS Sans Serif" BEGIN - PUSHBUTTON "&Close",12,7,72,50,14 - PUSHBUTTON "&Emergency save",13,155,72,68,14 - PUSHBUTTON "",15,243,72,55,14 - EDITTEXT 11,7,93,291,118,ES_MULTILINE | ES_READONLY | WS_VSCROLL | + PUSHBUTTON "&Close",12,7,80,50,14 + PUSHBUTTON "&Emergency save",13,155,80,68,14 + PUSHBUTTON "",15,243,80,55,14 + EDITTEXT 11,7,101,291,118,ES_MULTILINE | ES_READONLY | WS_VSCROLL | WS_HSCROLL | NOT WS_TABSTOP - LTEXT "",10,36,7,262,57 + LTEXT "",10,36,7,262,65 ICON 100,IDC_STATIC,9,9,20,20 END -- cgit v1.2.3-70-g09d2