On Linux show in crash message box the backtrace

[lyx.git] / src / frontends / qt4 / GuiApplication.cpp

diff --git a/src/frontends/qt4/GuiApplication.cpp b/src/frontends/qt4/GuiApplication.cpp
index 136b6e99fcbf8db5c57e556b7d8dcd390f740a00..f6c8433141131732be85ac94fa9ab5dc642c366c 100644 (file)
--- a/src/frontends/qt4/GuiApplication.cpp
+++ b/src/frontends/qt4/GuiApplication.cpp
@@ -70,6 +70,7 @@
 #include "support/Package.h"
 #include "support/PathChanger.h"
 #include "support/Systemcall.h"
+#include "support/TempFile.h"
 
 #ifdef Q_WS_MACX
 #include "support/AppleScript.h"
@@ -175,6 +176,25 @@ frontend::Application * createApplication(int & argc, char * argv[])
        return guiApp;
 }
 
+
+void setLocale()
+{
+       QLocale theLocale;
+       if (lyxrc.gui_language == "auto") {
+               theLocale = QLocale::system();
+       } else {
+               Language const * l = languages.getLanguage(lyxrc.gui_language);
+               string const code = l ? l->code() : string();
+               theLocale = QLocale(toqstr(code));
+       }
+       string const code = fromqstr(theLocale.name());
+       // Qt tries to outsmart us and transforms en_US to C.
+       Messages::guiLanguage((code == "C") ? "en_US" : code);
+       QLocale::setDefault(theLocale);
+       setlocale(LC_NUMERIC, "C");
+}
+
+
 namespace frontend {
 
 
@@ -832,8 +852,19 @@ public:
                case QEvent::Show:
                case QEvent::Hide:
                case QEvent::Resize:
+               case QEvent::UpdateRequest:
+               case QEvent::CursorChange:
+               case QEvent::ActionChanged:
+               case QEvent::EnabledChange:
+               case QEvent::SockAct:
+               case QEvent::Timer:
+               case QEvent::Paint:
+               case QEvent::ToolTipChange:
+               case QEvent::LayoutRequest:
+               case QEvent::MetaCall:
                        return QObject::eventFilter(obj, event);
                default:
+                       // FIXME Blocking all these events is a bad idea.
                        QKeyEvent *keyEvent = dynamic_cast<QKeyEvent*>(event);
                        if (keyEvent && keyEvent->key() == Qt::Key_Escape)
                                pressed_ = true;
@@ -946,6 +977,8 @@ GuiApplication::GuiApplication(int & argc, char ** argv)
        QCoreApplication::setOrganizationDomain("lyx.org");
        QCoreApplication::setApplicationName(lyx_package);
 
+       qsrand(QDateTime::currentDateTime().toTime_t());
+
        // Install translator for GUI elements.
        installTranslator(&d->qt_trans_);
 
@@ -1220,6 +1253,14 @@ bool GuiApplication::getStatus(FuncRequest const & cmd, FuncStatus & flag) const
                break;
        }
 
+       case LFUN_DIALOG_SHOW: {
+               string const name = cmd.getArg(0);
+               return name == "aboutlyx"
+                       || name == "prefs"
+                       || name == "texinfo"
+                       || name == "progress"
+                       || name == "compare";
+       }
 
        default:
                return false;
@@ -1551,8 +1592,12 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                validateCurrentView();
                // FIXME: create a new method shared with LFUN_HELP_OPEN.
                string const fname = to_utf8(cmd.argument());
-               if (d->views_.empty() || (!lyxrc.open_buffers_in_tabs
-                         && current_view_->documentBufferView() != 0)) {
+               bool const is_open = FileName::isAbsolute(fname)
+                       && theBufferList().getBuffer(FileName(fname));
+               if (d->views_.empty()
+                   || (!lyxrc.open_buffers_in_tabs
+                       && current_view_->documentBufferView() != 0
+                       && !is_open)) {
                        // We want the ui session to be saved per document and not per
                        // window number. The filename crc is a good enough identifier.
                        boost::crc_32_type crc;
@@ -1647,7 +1692,18 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                }
 
                actOnUpdatedPrefs(lyxrc_orig, lyxrc);
-               resetGui();
+
+               // If the request comes from the minibuffer, then we can't reset
+               // the GUI, since that would destory the minibuffer itself and
+               // cause a crash, since we are currently in one of the methods of 
+               // GuiCommandBuffer. See bug #8540. 
+               if (cmd.origin() != FuncRequest::COMMANDBUFFER)
+                       resetGui();
+               // else
+               //   FIXME Unfortunately, that leaves a bug here, since we cannot
+               //   reset the GUI in this case. If the changes to lyxrc affected the
+               //   UI, then, nothing would happen. This seems fairly unlikely, but 
+               //   it definitely is a bug.
 
                break;
        }
@@ -1892,6 +1948,24 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                lyxerr.setLevel(Debug::value(to_utf8(cmd.argument())));
                break;
 
+       case LFUN_DIALOG_SHOW: {
+               string const name = cmd.getArg(0);
+
+               if ( name == "aboutlyx"
+                       || name == "prefs"
+                       || name == "texinfo"
+                       || name == "progress"
+                       || name == "compare")
+               {
+                       // work around: on Mac OS the application
+                       // is not terminated when closing the last view.
+                       // Create a new one to be able to dispatch the
+                       // LFUN_DIALOG_SHOW to this view.
+                       if (current_view_ == 0)
+                               createView();
+               }
+       }
+
        default:
                // The LFUN must be for one of GuiView, BufferView, Buffer or Cursor;
                // let's try that:
@@ -2210,14 +2284,10 @@ void GuiApplication::exit(int status)
 
 void GuiApplication::setGuiLanguage()
 {
-       QString const default_language = toqstr(getGuiMessages().language());
-       LYXERR(Debug::LOCALE, "Trying to set default locale to: " << default_language);
-       QLocale const default_locale(default_language);
-       QLocale::setDefault(default_locale);
-
+       setLocale();
+       QLocale theLocale;
        // install translation file for Qt built-in dialogs
-       QString const language_name = QString("qt_") + default_locale.name();
-
+       QString const language_name = QString("qt_") + theLocale.name();
        // language_name can be short (e.g. qt_zh) or long (e.g. qt_zh_CN).
        // Short-named translator can be loaded from a long name, but not the
        // opposite. Therefore, long name should be used without truncation.
@@ -2231,7 +2301,7 @@ void GuiApplication::setGuiLanguage()
                        << language_name);
        }
 
-       switch (default_locale.language()) {
+       switch (theLocale.language()) {
        case QLocale::Arabic :
        case QLocale::Hebrew :
        case QLocale::Persian :
@@ -2758,14 +2828,14 @@ bool GuiApplication::readUIFile(QString const & name, bool include)
 
        if (retval == FormatMismatch) {
                LYXERR(Debug::FILES, "Converting ui file to format " << LFUN_FORMAT);
-               FileName const tempfile = FileName::tempName("convert_ui");
+               TempFile tmp("convertXXXXXX.ui");
+               FileName const tempfile = tmp.name();
                bool const success = prefs2prefs(ui_path, tempfile, true);
                if (!success) {
                        LYXERR0("Unable to convert " << ui_path.absFileName() <<
                                " to format " << LFUN_FORMAT << ".");
                } else {
                        retval = readUIFile(tempfile);
-                       tempfile.removeFile();
                }
        }