From dd2e73f3658be1b7a7f9c0433c0236dc28183c4a Mon Sep 17 00:00:00 2001 From: Stephan Witt Date: Mon, 24 Jan 2011 10:13:18 +0000 Subject: [PATCH] #7149 correct current_view_ or add checks for it for: LFUN_WINDOW_CLOSE,LFUN_SET_COLOR,LFUN_SERVER_GET_FILENAME and LFUN_BOOKMARK_GOTO git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@37313 a592a061-630c-0410-9148-cb99ea01b6c8 --- src/frontends/qt4/GuiApplication.cpp | 34 ++++++++++++++++++++-------- 1 file changed, 24 insertions(+), 10 deletions(-) diff --git a/src/frontends/qt4/GuiApplication.cpp b/src/frontends/qt4/GuiApplication.cpp index 51c2c0fcdf..b6cebbe043 100644 --- a/src/frontends/qt4/GuiApplication.cpp +++ b/src/frontends/qt4/GuiApplication.cpp @@ -1169,10 +1169,13 @@ void GuiApplication::gotoBookmark(unsigned int idx, bool openFile, dispatch(FuncRequest(LFUN_BOOKMARK_SAVE, "0")); // if the current buffer is not that one, switch to it. - BufferView * doc_bv = current_view_->documentBufferView(); + BufferView * doc_bv = current_view_ ? + current_view_->documentBufferView() : 0; if (!doc_bv || doc_bv->buffer().fileName() != tmp.filename) { if (switchToBuffer) { dispatch(FuncRequest(LFUN_BUFFER_SWITCH, file)); + if (!current_view_) + return; doc_bv = current_view_->documentBufferView(); } else return; @@ -1295,7 +1298,10 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr) // clear the last opened list, because // maybe this will end the session theSession().lastOpened().clear(); - current_view_->closeScheduled(); + // check for valid current_view_ + validateCurrentView(); + if (current_view_) + current_view_->closeScheduled(); break; case LFUN_LYX_QUIT: @@ -1418,8 +1424,9 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr) } if (!lcolor.setColor(lyx_name, x11_name)) { - current_view_->message( - bformat(_("Set-color \"%1$s\" failed " + if (current_view_) + current_view_->message( + bformat(_("Set-color \"%1$s\" failed " "- color is undefined or " "may not be redefined"), from_utf8(lyx_name))); @@ -1480,13 +1487,18 @@ void GuiApplication::dispatch(FuncRequest const & cmd, DispatchResult & dr) // --- lyxserver commands ---------------------------- case LFUN_SERVER_GET_FILENAME: { - LASSERT(current_view_ && current_view_->documentBufferView(), return); - docstring const fname = from_utf8( + if (current_view_ && current_view_->documentBufferView()) { + docstring const fname = from_utf8( current_view_->documentBufferView()->buffer().absFileName()); - dr.setMessage(fname); - LYXERR(Debug::INFO, "FNAME[" << fname << ']'); + dr.setMessage(fname); + LYXERR(Debug::INFO, "FNAME[" << fname << ']'); + } else { + dr.setMessage(docstring()); + LYXERR(Debug::INFO, "No current file for LFUN_SERVER_GET_FILENAME"); + } break; } + case LFUN_SERVER_NOTIFY: { docstring const dispatch_buffer = d->keyseq.print(KeySequence::Portable); dr.setMessage(dispatch_buffer); @@ -2041,6 +2053,8 @@ void GuiApplication::restoreGuiSession() LastOpenedSection::LastOpened const & lastopened = session.lastOpened().getfiles(); + validateCurrentView(); + FileName active_file; // do not add to the lastfile list since these files are restored from // last session, and should be already there (regular files), or should @@ -2062,7 +2076,7 @@ void GuiApplication::restoreGuiSession() // Restore last active buffer Buffer * buffer = theBufferList().getBuffer(active_file); - if (buffer) + if (buffer && current_view_) current_view_->setBuffer(buffer); // clear this list to save a few bytes of RAM @@ -2150,7 +2164,7 @@ bool GuiApplication::notify(QObject * receiver, QEvent * event) this->exit(1); case BufferException: { - if (!current_view_->documentBufferView()) + if (!current_view_ || !current_view_->documentBufferView()) return false; Buffer * buf = ¤t_view_->documentBufferView()->buffer(); docstring details = e.details_ + '\n'; -- 2.39.2