[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - sal/osl
Michael Stahl
mstahl at redhat.com
Wed Feb 1 13:35:15 UTC 2017
sal/osl/w32/dllentry.c | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
New commits:
commit 04b7757da617594347bcdfb1965d8129efce074a
Author: Michael Stahl <mstahl at redhat.com>
Date: Fri Jan 27 17:50:30 2017 +0100
sal: add SAL_NO_ASSERT_DIALOGS variable to redirect...
... the dialogs that pop up on assert() and abort() to stderr, which
should work better in CI builds.
Change-Id: I80ec95bae6cc21b8ac27f68ca93aa6c6cce707f4
(cherry picked from commit 2fc5a65a1808ce4304ea55f06c4587a2459b5d52)
Reviewed-on: https://gerrit.libreoffice.org/33719
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
diff --git a/sal/osl/w32/dllentry.c b/sal/osl/w32/dllentry.c
index 54133b5..96f26ca 100644
--- a/sal/osl/w32/dllentry.c
+++ b/sal/osl/w32/dllentry.c
@@ -23,6 +23,9 @@
#include <windows.h>
#ifdef _MSC_VER
#pragma warning(pop)
+#ifdef _DEBUG
+#include <crtdbg.h>
+#endif
#endif
#include <tlhelp32.h>
#include <systools/win32/uwinapi.h>
@@ -149,6 +152,21 @@ static BOOL WINAPI RawDllMain( HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvRe
{
case DLL_PROCESS_ATTACH:
{
+#ifdef _DEBUG
+ WCHAR buf[64];
+ DWORD const res = GetEnvironmentVariableW(L"SAL_NO_ASSERT_DIALOGS", buf, sizeof(buf));
+ if (res && res < sizeof(buf))
+ {
+ // disable the dialog on abort()
+ _CrtSetReportFile(_CRT_ERROR, _CRTDBG_FILE_STDERR);
+ _CrtSetReportMode(_CRT_ERROR, (_CRTDBG_MODE_DEBUG|_CRTDBG_MODE_FILE));
+ // not sure which assertions this affects
+ _CrtSetReportFile(_CRT_ASSERT, _CRTDBG_FILE_STDERR);
+ _CrtSetReportMode(_CRT_ASSERT, (_CRTDBG_MODE_DEBUG|_CRTDBG_MODE_FILE));
+ // disable the dialog on assert(false)
+ _set_error_mode(_OUT_TO_STDERR);
+ }
+#endif
#endif
#if OSL_DEBUG_LEVEL < 2
More information about the Libreoffice-commits
mailing list