Failed assertion throws runtime_error; message box moved to GUI main loop try..catch wrapper (#496)

Failed assertions were not properly handled in extant code. We launched a QMessageBox from a thread, which is unsafe according to the Qt docs, and actually failed (it could be repaired by shortening the error message, e.g. by ommitting the file name, but that would be quite fragile a solution).

Now, a failed assertion just throws a std::runtime_error. We don't use qFatal any longer. This allows us to remove the failedAssert function pointer and the QMessageHandler.

If catched inside the GUI, the failed assertion is displayed in the message frame. Otherwise, the exception is catched in the main program, which launches a QMessageBox within a fresh QApplication.

This leaves one specific case uncovered, namely: Failed assertions (and other exceptions) thrown in event handlers but outside specific try blocks.

App/main.cpp

@@ -36,11 +36,11 @@ int main(int argc, char* argv[])
     qRegisterMetaType<QVector<double>>("QVector<double>");
 
     QApplication app(argc, argv);
-    QApplication::setApplicationName("BornAgain");
-    QApplication::setApplicationVersion(GUI::Util::Path::getBornAgainVersionString());
-    QApplication::setOrganizationName("BornAgain");
+    app.setApplicationName("BornAgain");
+    app.setApplicationVersion(GUI::Util::Path::getBornAgainVersionString());
+    app.setOrganizationName("BornAgain");
 #ifndef Q_OS_MAC
-    QApplication::setWindowIcon(QIcon(":/images/BornAgain.ico"));
+    app.setWindowIcon(QIcon(":/images/BornAgain.ico"));
 #endif
 
     ApplicationSettings applicationSettings;
@@ -62,8 +62,9 @@ int main(int argc, char* argv[])
         win.loadProject(options.projectFile());
     win.show();
 
+    int ret = -1;
     try {
-        return QApplication::exec();
+        ret = app.exec();
     } catch (const std::exception& ex) {
         QMessageBox msgbox(QMessageBox::Critical, "BornAgain: fatal bug",
                            QString("Sorry, you encountered a fatal bug.\n"
@@ -72,5 +73,7 @@ int main(int argc, char* argv[])
                            + ex.what() + "\n",
                            QMessageBox::Ok, QApplication::activeWindow());
         msgbox.exec();
+        return 1;
     }
+    return ret;
 }