From fe71cee15fca25027f7ebcde30734b46c3030e7b Mon Sep 17 00:00:00 2001 From: Juozas Date: Sat, 4 Sep 2021 13:55:41 +0300 Subject: [PATCH] Remove crash reporting from help menu This commit removes unused crash reporting menu items from the help menu as the code related to it were already removed from the code tenacity. Signed-off-by: Juozas --- src/menus/HelpMenus.cpp | 34 ---------------------------------- 1 file changed, 34 deletions(-) diff --git a/src/menus/HelpMenus.cpp b/src/menus/HelpMenus.cpp index 5e50d4845..bd73f1ea4 100644 --- a/src/menus/HelpMenus.cpp +++ b/src/menus/HelpMenus.cpp @@ -347,31 +347,6 @@ void OnShowLog( const CommandContext &context ) } } -#ifdef IS_ALPHA -void OnSegfault(const CommandContext &) -{ - unsigned *p = nullptr; - *p = 0xDEADBEEF; -} - -void OnException(const CommandContext &) -{ - // Throw an exception that can be caught only as (...) - // The intent is to exercise detection of unhandled exceptions by the - // crash reporter - struct Unique{}; - throw Unique{}; -} - -void OnAssertion(const CommandContext &) -{ - // We don't use assert() much directly, but Breakpad does detect it - // This may crash the program only in debug builds - // See also wxSetAssertHandler, and wxApp::OnAssertFailure() - assert(false); -} -#endif - void OnMenuTree(const CommandContext &context) { auto &project = context.project; @@ -534,15 +509,6 @@ BaseItemSharedPtr HelpMenu() // alpha-only items don't need to internationalize, so use // Verbatim for labels - Command( wxT("RaiseSegfault"), Verbatim("Test segfault report"), - FN(OnSegfault), AlwaysEnabledFlag ), - - Command( wxT("ThrowException"), Verbatim("Test exception report"), - FN(OnException), AlwaysEnabledFlag ), - - Command( wxT("ViolateAssertion"), Verbatim("Test assertion report"), - FN(OnAssertion), AlwaysEnabledFlag ), - // Menu explorer. Perhaps this should become a macro command Command( wxT("MenuTree"), Verbatim("Menu Tree..."), FN(OnMenuTree),