From 2e934fc5f85b9e00ea4308000ea0674cd64c376d Mon Sep 17 00:00:00 2001 From: Richard Heck Date: Tue, 3 Oct 2017 17:28:35 -0400 Subject: [PATCH] Remove updateInfo() calls in favor of doing the relevant work in updateBuffer(). This is in response to a reported crash. See https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg202217.html Commit af381a2f addressed the crash, and is worth doing anyway. But this also makes sense. Also adds a flag to tell us whether we need to recalculate the relevant information. In some cases, there is no need to do so more than once. (Note that, with the existing code, we *never* recalculate, which would have given rise to bugs.) --- src/factory.cpp | 1 - src/insets/InsetInfo.cpp | 47 +++++++++++++++++++++++++++++++++++----- src/insets/InsetInfo.h | 6 +++-- 3 files changed, 46 insertions(+), 8 deletions(-) diff --git a/src/factory.cpp b/src/factory.cpp index 72019cd6f7..3937a9e7f8 100644 --- a/src/factory.cpp +++ b/src/factory.cpp @@ -263,7 +263,6 @@ Inset * createInsetHelper(Buffer * buf, FuncRequest const & cmd) case LFUN_INFO_INSERT: { InsetInfo * inset = new InsetInfo(buf, to_utf8(cmd.argument())); - inset->updateInfo(); return inset; } diff --git a/src/insets/InsetInfo.cpp b/src/insets/InsetInfo.cpp index 42e0f6cda2..d52f887031 100644 --- a/src/insets/InsetInfo.cpp +++ b/src/insets/InsetInfo.cpp @@ -92,7 +92,8 @@ NameTranslator const & nameTranslator() InsetInfo::InsetInfo(Buffer * buf, string const & name) - : InsetCollapsable(buf), type_(UNKNOWN_INFO), name_() + : InsetCollapsable(buf), initialized_(false), + type_(UNKNOWN_INFO), name_() { setInfo(name); status_ = Collapsed; @@ -147,7 +148,6 @@ void InsetInfo::read(Lexer & lex) _("Missing \\end_inset at this point."), from_utf8(token)); } - updateInfo(); } @@ -247,6 +247,7 @@ void InsetInfo::doDispatch(Cursor & cur, FuncRequest & cmd) case LFUN_INSET_MODIFY: cur.recordUndo(); setInfo(to_utf8(cmd.argument())); + initialized_ = false; break; case LFUN_INSET_COPY_AS: { @@ -278,7 +279,6 @@ void InsetInfo::setInfo(string const & name) string type; name_ = trim(split(name, type, ' ')); type_ = nameTranslator().find(type); - updateInfo(); } @@ -295,16 +295,21 @@ void InsetInfo::setText(docstring const & str) } -void InsetInfo::updateInfo() -{ +void InsetInfo::updateBuffer(ParIterator const & it, UpdateType utype) { BufferParams const & bp = buffer().params(); switch (type_) { case UNKNOWN_INFO: error("Unknown Info: %1$s"); + initialized_ = false; break; case SHORTCUT_INFO: case SHORTCUTS_INFO: { + // only need to do this once. + if (initialized_) + break; + // and we will not keep trying if we fail + initialized_ = true; FuncRequest const func = lyxaction.lookupFunc(name_); if (func.action() == LFUN_UNKNOWN_ACTION) { error("Unknown action %1$s"); @@ -323,11 +328,15 @@ void InsetInfo::updateInfo() break; } case LYXRC_INFO: { + // this information could actually change, if the preferences are changed, + // so we will recalculate each time through. ostringstream oss; if (name_.empty()) { setText(_("undefined")); break; } + // FIXME this uses the serialization mechanism to get the info + // we want, which i guess works but is a bit strange. lyxrc.write(oss, true, name_); string result = oss.str(); if (result.size() < 2) { @@ -351,20 +360,33 @@ void InsetInfo::updateInfo() break; } case PACKAGE_INFO: + // only need to do this once. + if (initialized_) + break; // check in packages.lst setText(LaTeXFeatures::isAvailable(name_) ? _("yes") : _("no")); + initialized_ = true; break; case TEXTCLASS_INFO: { + // only need to do this once. + if (initialized_) + break; // name_ is the class name LayoutFileList const & list = LayoutFileList::get(); bool available = false; if (list.haveClass(name_)) available = list[name_].isTeXClassAvailable(); setText(available ? _("yes") : _("no")); + initialized_ = true; break; } case MENU_INFO: { + // only need to do this once. + if (initialized_) + break; + // and we will not keep trying if we fail + initialized_ = true; docstring_list names; FuncRequest const func = lyxaction.lookupFunc(name_); if (func.action() == LFUN_UNKNOWN_ACTION) { @@ -406,6 +428,11 @@ void InsetInfo::updateInfo() break; } case ICON_INFO: { + // only need to do this once. + if (initialized_) + break; + // and we will not keep trying if we fail + initialized_ = true; FuncRequest func = lyxaction.lookupFunc(name_); docstring icon_name = frontend::Application::iconName(func, true); // FIXME: We should use the icon directly instead of @@ -451,6 +478,7 @@ void InsetInfo::updateInfo() break; } case BUFFER_INFO: { + // this could all change, so we will recalculate each time if (name_ == "name") { setText(from_utf8(buffer().fileName().onlyFileName())); break; @@ -464,8 +492,12 @@ void InsetInfo::updateInfo() break; } + //////////////////////////////////////////////////////////////// // everything that follows is for version control. // nothing that isn't version control should go below this line. + + // this information could change, in principle, so we will + // recalculate each time through if (!buffer().lyxvc().inUse()) { setText(_("No version control")); break; @@ -489,10 +521,15 @@ void InsetInfo::updateInfo() break; } case LYX_INFO: + // only need to do this once. + if (initialized_) + break; if (name_ == "version") setText(from_ascii(lyx_version)); + initialized_ = true; break; } + InsetCollapsable::updateBuffer(it, utype); } diff --git a/src/insets/InsetInfo.h b/src/insets/InsetInfo.h index 96ba22cf12..8ab6f826ca 100644 --- a/src/insets/InsetInfo.h +++ b/src/insets/InsetInfo.h @@ -127,8 +127,8 @@ public: void doDispatch(Cursor & cur, FuncRequest & cmd); /// void setInfo(std::string const & info); - /// update info_ and text - void updateInfo(); + /// + void updateBuffer(ParIterator const & it, UpdateType utype); /// docstring toolTip(BufferView const & bv, int x, int y) const; /// @@ -148,6 +148,8 @@ private: // make sure that the other version of setText is still available. using InsetCollapsable::setText; /// + bool initialized_; + /// info_type type_; /// std::string name_; -- 2.39.5