From 691dcb8ae1eecec2ca957eab88059eb0f3a906b3 Mon Sep 17 00:00:00 2001 From: John Preston Date: Tue, 4 May 2021 00:02:24 +0400 Subject: [PATCH] Fix build on macOS and Linux. --- Telegram/SourceFiles/core/crash_reports.cpp | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/Telegram/SourceFiles/core/crash_reports.cpp b/Telegram/SourceFiles/core/crash_reports.cpp index 39febb8d73..022c6a14c9 100644 --- a/Telegram/SourceFiles/core/crash_reports.cpp +++ b/Telegram/SourceFiles/core/crash_reports.cpp @@ -16,8 +16,6 @@ https://github.com/telegramdesktop/tdesktop/blob/master/LEGAL #include #ifndef DESKTOP_APP_DISABLE_CRASH_REPORTS - -// see https://blog.inventic.eu/2012/08/qt-and-google-breakpad/ #ifdef Q_OS_WIN #pragma warning(push) @@ -25,11 +23,13 @@ https://github.com/telegramdesktop/tdesktop/blob/master/LEGAL #include "client/windows/handler/exception_handler.h" #pragma warning(pop) -#elif defined Q_OS_MAC // Q_OS_WIN +#elif defined Q_OS_UNIX // Q_OS_WIN #include -#include #include + +#ifdef Q_OS_MAC + #include #include @@ -39,16 +39,13 @@ https://github.com/telegramdesktop/tdesktop/blob/master/LEGAL #include "client/crashpad_client.h" #endif // else for MAC_USE_BREAKPAD -#elif defined Q_OS_UNIX // Q_OS_MAC - -#include -#include -#include +#else // Q_OS_MAC #include "client/linux/handler/exception_handler.h" -#endif // Q_OS_UNIX +#endif // Q_OS_MAC +#endif // Q_OS_WIN #endif // !DESKTOP_APP_DISABLE_CRASH_REPORTS namespace CrashReports { @@ -203,7 +200,7 @@ struct sigaction OldSigActions[32]/* = { 0 }*/; void RestoreSignalHandlers() { for (const auto signum : HandledSignals) { - sigaction(signum, &OldSigActions[signal], nullptr); + sigaction(signum, &OldSigActions[signum], nullptr); } } @@ -446,7 +443,7 @@ Status Restart() { sigact.sa_flags = SA_NODEFER | SA_RESETHAND | SA_SIGINFO; for (const auto signum : HandledSignals) { - sigaction(signum, &sigact, &OldSigActions[signal]); + sigaction(signum, &sigact, &OldSigActions[signum]); } #else // !Q_OS_WIN for (const auto signum : HandledSignals) {