X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FMenuBackend.C;h=294599d1cf3e0042c23799418b6aea525c7b3c9b;hb=701b99ecd70ac472aa53c8b2317af44def4f9670;hp=c0903bfda7e342cc9da5c9bbf986b4ae44427000;hpb=5d3718cad2a2ef6a4d6a495054ab0705ba27b6b5;p=lyx.git diff --git a/src/MenuBackend.C b/src/MenuBackend.C index c0903bfda7..294599d1cf 100644 --- a/src/MenuBackend.C +++ b/src/MenuBackend.C @@ -1,91 +1,113 @@ -/* This file is part of - * ====================================================== +/** + * \file MenuBackend.C + * This file is part of LyX, the document processor. + * Licence details can be found in the file COPYING. * - * LyX, The Document Processor + * \author Asger Alstrup + * \author Lars Gullik Bjønnes + * \author Jean-Marc Lasgouttes + * \author André Pönitz + * \author Dekel Tsur + * \author Martin Vermeer * - * Copyright 1995 Matthias Ettrich - * Copyright 1995-2001 The LyX Team. - * - * - * ====================================================== */ + * Full author contact details are available in file CREDITS. + */ #include -#include #include "MenuBackend.h" -#include "lyxlex.h" -#include "LyXAction.h" + +#include "BranchList.h" +#include "buffer.h" +#include "bufferlist.h" +#include "bufferparams.h" +#include "CutAndPaste.h" #include "debug.h" +#include "exporter.h" +#include "Floating.h" +#include "FloatList.h" +#include "format.h" #include "gettext.h" +#include "importer.h" #include "kbmap.h" #include "lastfiles.h" -#include "lyxfunc.h" +#include "LyXAction.h" #include "lyx_main.h" // for lastfiles -#include "bufferlist.h" -#include "buffer.h" -#include "converter.h" -#include "exporter.h" -#include "importer.h" -#include "FloatList.h" +#include "lyxfunc.h" +#include "lyxlex.h" #include "toc.h" + #include "frontends/LyXView.h" -#include "support/LAssert.h" + #include "support/filetools.h" -#include "support/lyxfunctional.h" #include "support/lstrings.h" +#include "support/tostr.h" -extern BufferList bufferlist; -extern boost::scoped_ptr toplevel_keymap; +#include + +#include + +using lyx::support::compare_ascii_no_case; +using lyx::support::contains; +using lyx::support::MakeDisplayPath; +using lyx::support::token; + +using boost::bind; +using std::auto_ptr; using std::endl; -using std::vector; -using std::max; -using std::pair; +using std::equal_to; using std::find_if; +using std::max; using std::sort; +using std::string; +using std::vector; + + +extern BufferList bufferlist; +extern boost::scoped_ptr toplevel_keymap; + +namespace { + +class MenuNamesEqual : public std::unary_function { +public: + MenuNamesEqual(string const & name) + : name_(name) {} + bool operator()(Menu const & menu) const + { + return menu.name() == name_; + } +private: + string name_; +}; + +} // namespace anon + // This is the global menu definition MenuBackend menubackend; +MenuItem::MenuItem(Kind kind) + : kind_(kind), optional_(false) +{} + + MenuItem::MenuItem(Kind kind, string const & label, - string const & command, bool optional) - : kind_(kind), label_(label), optional_(optional) + string const & submenu, bool optional) + : kind_(kind), label_(label), + submenuname_(submenu), optional_(optional) { - switch (kind) { - case Separator: - case Documents: - case Lastfiles: - case Toc: - case ViewFormats: - case UpdateFormats: - case ExportFormats: - case ImportFormats: - case FloatListInsert: - case FloatInsert: - break; - case Command: - action_ = lyxaction.LookupFunc(command); - - if (action_ == LFUN_UNKNOWN_ACTION) { - lyxerr << "MenuItem(): LyX command `" - << command << "' does not exist." << endl; - } - if (optional_) - lyxerr[Debug::GUI] << "Optional item " - << command << endl; - break; - case Submenu: - submenuname_ = command; - break; - } + BOOST_ASSERT(kind == Submenu); } -MenuItem::MenuItem(Kind kind, string const & label, int action, bool optional) - : kind_(kind), label_(label), action_(action), submenuname_(), - optional_(optional) -{} +MenuItem::MenuItem(Kind kind, string const & label, + FuncRequest const & func, bool optional) + : kind_(kind), label_(label), func_(func), optional_(optional) +{ + func_.origin = FuncRequest::UI; +} MenuItem::~MenuItem() @@ -109,19 +131,26 @@ string const MenuItem::shortcut() const return token(label_, '|', 1); } + string const MenuItem::binding() const { if (kind_ != Command) return string(); - + // Get the keys bound to this action, but keep only the // first one later - string bindings = toplevel_keymap->findbinding(action_); + kb_keymap::Bindings bindings = toplevel_keymap->findbindings(func_); - if (!bindings.empty()) { - return bindings.substr(1, bindings.find(']') - 1); - } else + if (bindings.size()) { + return bindings.begin()->print(); + } else { + lyxerr[Debug::KBMAP] + << "No binding for " + << lyxaction.getActionName(func_.action) + << '(' << func_.argument << ')' << endl; return string(); + } + } @@ -131,49 +160,51 @@ Menu & Menu::add(MenuItem const & i, LyXView const * view) items_.push_back(i); return *this; } - + switch (i.kind()) { - case MenuItem::Command: - { + + case MenuItem::Command: { FuncStatus status = - view->getLyXFunc().getStatus(i.action()); + view->getLyXFunc().getStatus(i.func()); if (status.unknown() - || (status.disabled() && i.optional())) + || (!status.enabled() && i.optional())) break; items_.push_back(i); items_.back().status(status); break; } - case MenuItem::Submenu: - { + + case MenuItem::Submenu: { if (i.submenu()) { - bool disabled = true; + bool enabled = false; for (const_iterator cit = i.submenu()->begin(); cit != i.submenu()->end(); ++cit) { if ((cit->kind() == MenuItem::Command || cit->kind() == MenuItem::Submenu) - && !cit->status().disabled()) { - disabled = false; + && cit->status().enabled()) { + enabled = true; break; } } - if (!disabled || !i.optional()) { + if (enabled || !i.optional()) { items_.push_back(i); - items_.back().status().disabled(disabled); + items_.back().status().enabled(enabled); } } else items_.push_back(i); break; } + case MenuItem::Separator: if (!items_.empty() && items_.back().kind() != MenuItem::Separator) items_.push_back(i); break; + default: items_.push_back(i); - } + } return *this; } @@ -183,7 +214,9 @@ Menu & Menu::read(LyXLex & lex) { enum Menutags { md_item = 1, + md_branches, md_documents, + md_charstyles, md_endmenu, md_exportformats, md_importformats, @@ -197,10 +230,13 @@ Menu & Menu::read(LyXLex & lex) md_viewformats, md_floatlistinsert, md_floatinsert, + md_pasterecent, md_last }; struct keyword_item menutags[md_last - 1] = { + { "branches", md_branches }, + { "charstyles", md_charstyles }, { "documents", md_documents }, { "end", md_endmenu }, { "exportformats", md_exportformats }, @@ -211,6 +247,7 @@ Menu & Menu::read(LyXLex & lex) { "lastfiles", md_lastfiles }, { "optitem", md_optitem }, { "optsubmenu", md_optsubmenu }, + { "pasterecent", md_pasterecent }, { "separator", md_separator }, { "submenu", md_submenu }, { "toc", md_toc }, @@ -235,8 +272,8 @@ Menu & Menu::read(LyXLex & lex) string const name = _(lex.getString()); lex.next(true); string const command = lex.getString(); - add(MenuItem(MenuItem::Command, name, - command, optional)); + FuncRequest func = lyxaction.lookupFunc(command); + add(MenuItem(MenuItem::Command, name, func, optional)); optional = false; break; } @@ -249,6 +286,10 @@ Menu & Menu::read(LyXLex & lex) add(MenuItem(MenuItem::Lastfiles)); break; + case md_charstyles: + add(MenuItem(MenuItem::CharStyles)); + break; + case md_documents: add(MenuItem(MenuItem::Documents)); break; @@ -281,6 +322,14 @@ Menu & Menu::read(LyXLex & lex) add(MenuItem(MenuItem::FloatInsert)); break; + case md_pasterecent: + add(MenuItem(MenuItem::PasteRecent)); + break; + + case md_branches: + add(MenuItem(MenuItem::Branches)); + break; + case md_optsubmenu: optional = true; // fallback to md_submenu @@ -310,10 +359,24 @@ Menu & Menu::read(LyXLex & lex) } +MenuItem const & Menu::operator[](size_type i) const +{ + return items_[i]; +} + + +bool Menu::hasFunc(FuncRequest const & func) const +{ + return find_if(begin(), end(), + bind(std::equal_to(), + bind(&MenuItem::func, _1), + func)) != end(); +} + void Menu::checkShortcuts() const { // This is a quadratic algorithm, but we do not care because - // it is used for debugging only. + // menus are short enough for (const_iterator it1 = begin(); it1 != end(); ++it1) { string shortcut = it1->shortcut(); if (shortcut.empty()) @@ -322,7 +385,7 @@ void Menu::checkShortcuts() const lyxerr << "Menu warning: menu entry \"" << it1->label() << "\" does not contain shortcut `" - << shortcut << '\'' << endl; + << shortcut << "'." << endl; for (const_iterator it2 = begin(); it2 != it1 ; ++it2) { if (!compare_ascii_no_case(it2->shortcut(), shortcut)) { lyxerr << "Menu warning: menu entries " @@ -336,6 +399,13 @@ void Menu::checkShortcuts() const } +void MenuBackend::specialMenu(string const &name) +{ + if (hasMenu(name)) + specialmenu_ = &getMenu(name); +} + + namespace { class compare_format { @@ -358,18 +428,17 @@ string const limit_string_length(string const & str) void expandLastfiles(Menu & tomenu, LyXView const * view) { + LastFiles const & lastfiles = LyX::cref().lastfiles(); + int ii = 1; - LastFiles::const_iterator lfit = lastfiles->begin(); - LastFiles::const_iterator end = lastfiles->end(); + LastFiles::const_iterator lfit = lastfiles.begin(); + LastFiles::const_iterator end = lastfiles.end(); for (; lfit != end && ii < 10; ++lfit, ++ii) { string const label = tostr(ii) + ". " + MakeDisplayPath((*lfit), 30) + '|' + tostr(ii); - int const action = lyxaction. - getPseudoAction(LFUN_FILE_OPEN, - (*lfit)); - tomenu.add(MenuItem(MenuItem::Command, label, action), view); + tomenu.add(MenuItem(MenuItem::Command, label, FuncRequest(LFUN_FILE_OPEN, (*lfit))), view); } } @@ -380,7 +449,7 @@ void expandDocuments(Menu & tomenu, LyXView const * view) if (names.empty()) { tomenu.add(MenuItem(MenuItem::Command, _("No Documents Open!"), - LFUN_NOACTION), view); + FuncRequest(LFUN_NOACTION)), view); return; } @@ -388,12 +457,10 @@ void expandDocuments(Menu & tomenu, LyXView const * view) Strings::const_iterator docit = names.begin(); Strings::const_iterator end = names.end(); for (; docit != end; ++docit, ++ii) { - int const action = - lyxaction.getPseudoAction(LFUN_SWITCHBUFFER, *docit); - string label = MakeDisplayPath(*docit, 30); + string label = MakeDisplayPath(*docit, 20); if (ii < 10) label = tostr(ii) + ". " + label + '|' + tostr(ii); - tomenu.add(MenuItem(MenuItem::Command, label, action), view); + tomenu.add(MenuItem(MenuItem::Command, label, FuncRequest(LFUN_SWITCHBUFFER, *docit)), view); } } @@ -402,7 +469,8 @@ void expandFormats(MenuItem::Kind kind, Menu & tomenu, LyXView const * view) { if (!view->buffer() && kind != MenuItem::ImportFormats) { tomenu.add(MenuItem(MenuItem::Command, - _("No Documents Open!"), LFUN_NOACTION), + _("No Documents Open!"), + FuncRequest(LFUN_NOACTION)), view); return; } @@ -417,15 +485,15 @@ void expandFormats(MenuItem::Kind kind, Menu & tomenu, LyXView const * view) action = LFUN_IMPORT; break; case MenuItem::ViewFormats: - formats = Exporter::GetExportableFormats(view->buffer(), true); + formats = Exporter::GetExportableFormats(*view->buffer(), true); action = LFUN_PREVIEW; break; case MenuItem::UpdateFormats: - formats = Exporter::GetExportableFormats(view->buffer(), true); + formats = Exporter::GetExportableFormats(*view->buffer(), true); action = LFUN_UPDATE; break; default: - formats = Exporter::GetExportableFormats(view->buffer(), false); + formats = Exporter::GetExportableFormats(*view->buffer(), false); action = LFUN_EXPORT; } sort(formats.begin(), formats.end(), compare_format()); @@ -451,9 +519,9 @@ void expandFormats(MenuItem::Kind kind, Menu & tomenu, LyXView const * view) } if (!(*fit)->shortcut().empty()) label += '|' + (*fit)->shortcut(); - int const action2 = lyxaction. - getPseudoAction(action, (*fit)->name()); - tomenu.add(MenuItem(MenuItem::Command, label, action2), + + tomenu.add(MenuItem(MenuItem::Command, label, + FuncRequest(action, (*fit)->name())), view); } } @@ -463,20 +531,21 @@ void expandFloatListInsert(Menu & tomenu, LyXView const * view) { if (!view->buffer()) { tomenu.add(MenuItem(MenuItem::Command, - _("No Documents Open!"), LFUN_NOACTION), + _("No Documents Open!"), + FuncRequest(LFUN_NOACTION)), view); return; } FloatList const & floats = - view->buffer()->params.getLyXTextClass().floats(); + view->buffer()->params().getLyXTextClass().floats(); FloatList::const_iterator cit = floats.begin(); FloatList::const_iterator end = floats.end(); for (; cit != end; ++cit) { - int const action = lyxaction - .getPseudoAction(LFUN_FLOAT_LIST, cit->second.type()); tomenu.add(MenuItem(MenuItem::Command, - _(cit->second.listName()), action), + _(cit->second.listName()), + FuncRequest(LFUN_FLOAT_LIST, + cit->second.type())), view); } } @@ -486,62 +555,87 @@ void expandFloatInsert(Menu & tomenu, LyXView const * view) { if (!view->buffer()) { tomenu.add(MenuItem(MenuItem::Command, - _("No Documents Open!"), LFUN_NOACTION), + _("No Documents Open!"), + FuncRequest(LFUN_NOACTION)), view); return; } FloatList const & floats = - view->buffer()->params.getLyXTextClass().floats(); + view->buffer()->params().getLyXTextClass().floats(); FloatList::const_iterator cit = floats.begin(); FloatList::const_iterator end = floats.end(); for (; cit != end; ++cit) { // normal float - int const action = - lyxaction.getPseudoAction(LFUN_INSET_FLOAT, - cit->second.type()); string const label = _(cit->second.name()); - tomenu.add(MenuItem(MenuItem::Command, label, action), + tomenu.add(MenuItem(MenuItem::Command, label, + FuncRequest(LFUN_INSET_FLOAT, + cit->second.type())), + view); + } +} + + +void expandCharStyleInsert(Menu & tomenu, LyXView const * view) +{ + if (!view->buffer()) { + tomenu.add(MenuItem(MenuItem::Command, + _("No Documents Open!"), + FuncRequest(LFUN_NOACTION)), view); + return; + } + CharStyles & charstyles = + view->buffer()->params().getLyXTextClass().charstyles(); + CharStyles::iterator cit = charstyles.begin(); + CharStyles::iterator end = charstyles.end(); + for (; cit != end; ++cit) { + string const label = cit->name; + tomenu.add(MenuItem(MenuItem::Command, label, + FuncRequest(LFUN_INSERT_CHARSTYLE, + cit->name)), view); } } Menu::size_type const max_number_of_items = 25; -void expandToc2(Menu & tomenu, toc::Toc const & toc_list, - toc::Toc::size_type from, toc::Toc::size_type to, int depth) +void expandToc2(Menu & tomenu, + lyx::toc::Toc const & toc_list, + lyx::toc::Toc::size_type from, + lyx::toc::Toc::size_type to, int depth) { int shortcut_count = 0; if (to - from <= max_number_of_items) { - for (toc::Toc::size_type i = from; i < to; ++i) { - int const action = toc_list[i].action(); + for (lyx::toc::Toc::size_type i = from; i < to; ++i) { string label(4 * max(0, toc_list[i].depth - depth),' '); label += limit_string_length(toc_list[i].str); if (toc_list[i].depth == depth - && ++shortcut_count <= 9) { - label += '|' + tostr(shortcut_count); + && shortcut_count < 9) { + if (label.find(tostr(shortcut_count + 1)) != string::npos) + label += '|' + tostr(++shortcut_count); } - tomenu.add(MenuItem(MenuItem::Command, label, action)); + tomenu.add(MenuItem(MenuItem::Command, label, + FuncRequest(toc_list[i].action()))); } } else { - toc::Toc::size_type pos = from; + lyx::toc::Toc::size_type pos = from; while (pos < to) { - toc::Toc::size_type new_pos = pos + 1; + lyx::toc::Toc::size_type new_pos = pos + 1; while (new_pos < to && toc_list[new_pos].depth > depth) ++new_pos; - int const action = toc_list[pos].action(); string label(4 * max(0, toc_list[pos].depth - depth), ' '); label += limit_string_length(toc_list[pos].str); if (toc_list[pos].depth == depth && - ++shortcut_count <= 9) - label += '|' + tostr(shortcut_count); - + shortcut_count < 9) { + if (label.find(tostr(shortcut_count + 1)) != string::npos) + label += '|' + tostr(++shortcut_count); + } if (new_pos == pos + 1) { tomenu.add(MenuItem(MenuItem::Command, - label, action)); + label, FuncRequest(toc_list[pos].action()))); } else { MenuItem item(MenuItem::Submenu, label); item.submenu(new Menu); @@ -564,32 +658,34 @@ void expandToc(Menu & tomenu, LyXView const * view) if (!view->buffer()) { tomenu.add(MenuItem(MenuItem::Command, - _("No Documents Open!"), LFUN_NOACTION), + _("No Documents Open!"), + FuncRequest(LFUN_NOACTION)), view); return; } - toc::TocList toc_list = toc::getTocList(view->buffer()); - toc::TocList::const_iterator cit = toc_list.begin(); - toc::TocList::const_iterator end = toc_list.end(); + lyx::toc::TocList toc_list = lyx::toc::getTocList(*view->buffer()); + lyx::toc::TocList::const_iterator cit = toc_list.begin(); + lyx::toc::TocList::const_iterator end = toc_list.end(); for (; cit != end; ++cit) { // Handle this later if (cit->first == "TOC") continue; // All the rest is for floats - Menu * menu = new Menu; - toc::Toc::const_iterator ccit = cit->second.begin(); - toc::Toc::const_iterator eend = cit->second.end(); + auto_ptr menu(new Menu); + lyx::toc::Toc::const_iterator ccit = cit->second.begin(); + lyx::toc::Toc::const_iterator eend = cit->second.end(); for (; ccit != eend; ++ccit) { string const label = limit_string_length(ccit->str); menu->add(MenuItem(MenuItem::Command, - label, ccit->action())); + label, + FuncRequest(ccit->action()))); } string const & floatName = cit->first; // Is the _(...) really needed here? (Lgb) MenuItem item(MenuItem::Submenu, _(floatName)); - item.submenu(menu); + item.submenu(menu.release()); tomenu.add(item); } @@ -597,7 +693,8 @@ void expandToc(Menu & tomenu, LyXView const * view) cit = toc_list.find("TOC"); if (cit == end) { tomenu.add(MenuItem(MenuItem::Command, - _("No Table of contents")), + _("No Table of contents"), + FuncRequest()), view); } else { expandToc2(tomenu, cit->second, 0, cit->second.size(), 0); @@ -605,6 +702,45 @@ void expandToc(Menu & tomenu, LyXView const * view) } +void expandPasteRecent(Menu & tomenu, LyXView const * view) +{ + if (!view || !view->buffer()) + return; + + vector const sel = + lyx::cap::availableSelections(*view->buffer()); + + vector::const_iterator cit = sel.begin(); + vector::const_iterator end = sel.end(); + + for (unsigned int index = 0; cit != end; ++cit, ++index) { + tomenu.add(MenuItem(MenuItem::Command, *cit, + FuncRequest(LFUN_PASTE, tostr(index)))); + } +} + + +void expandBranches(Menu & tomenu, LyXView const * view) +{ + if (!view || !view->buffer()) + return; + + BufferParams const & params = view->buffer()->params(); + + std::list::const_iterator cit = params.branchlist().begin(); + std::list::const_iterator end = params.branchlist().end(); + + for (int ii = 1; cit != end; ++cit, ++ii) { + string label = cit->getBranch(); + if (ii < 10) + label = tostr(ii) + ". " + label + "|" + tostr(ii); + tomenu.add(MenuItem(MenuItem::Command, label, + FuncRequest(LFUN_INSERT_BRANCH, + cit->getBranch())), view); + } +} + + } // namespace anon @@ -629,6 +765,10 @@ void MenuBackend::expand(Menu const & frommenu, Menu & tomenu, expandFormats(cit->kind(), tomenu, view); break; + case MenuItem::CharStyles: + expandCharStyleInsert(tomenu, view); + break; + case MenuItem::FloatListInsert: expandFloatListInsert(tomenu, view); break; @@ -637,6 +777,14 @@ void MenuBackend::expand(Menu const & frommenu, Menu & tomenu, expandFloatInsert(tomenu, view); break; + case MenuItem::PasteRecent: + expandPasteRecent(tomenu, view); + break; + + case MenuItem::Branches: + expandBranches(tomenu, view); + break; + case MenuItem::Toc: expandToc(tomenu, view); break; @@ -650,8 +798,14 @@ void MenuBackend::expand(Menu const & frommenu, Menu & tomenu, } break; - default: + case MenuItem::Separator: tomenu.add(*cit, view); + break; + + case MenuItem::Command: + if (!specialmenu_ + || !specialmenu_->hasFunc(cit->func())) + tomenu.add(*cit, view); } } @@ -661,16 +815,7 @@ void MenuBackend::expand(Menu const & frommenu, Menu & tomenu, tomenu.items_.pop_back(); // Check whether the shortcuts are unique - if (lyxerr.debugging(Debug::GUI)) - tomenu.checkShortcuts(); -} - - -bool Menu::hasSubmenu(string const & name) const -{ - return find_if(begin(), end(), - lyx::compare_memfun(&MenuItem::submenuname, - name)) != end(); + tomenu.checkShortcuts(); } @@ -731,51 +876,6 @@ void MenuBackend::read(LyXLex & lex) } -void MenuBackend::defaults() -{ - menulist_.clear(); - - lyxerr[Debug::GUI] << "MenuBackend::defaults: using default values" - << endl; - - Menu file("file"); - file - .add(MenuItem(MenuItem::Command, _("New...|N"), "buffer-new")) - .add(MenuItem(MenuItem::Command, _("Open...|O"), "file-open")) - .add(MenuItem(MenuItem::Submenu, _("Import|I"), "import")) - .add(MenuItem(MenuItem::Command, _("Quit|Q"), "lyx-quit")) - .add(MenuItem(MenuItem::Separator)) - .add(MenuItem(MenuItem::Lastfiles)); - add(file); - - Menu import("import"); - import - .add(MenuItem(MenuItem::Command, - _("LaTeX...|L"), "buffer-import latex")) - .add(MenuItem(MenuItem::Command, - _("LinuxDoc...|L"), "buffer-import linuxdoc")); - add(import); - - Menu edit("edit"); - edit - .add(MenuItem(MenuItem::Command, _("Cut"), "cut")) - .add(MenuItem(MenuItem::Command, _("Copy"), "copy")) - .add(MenuItem(MenuItem::Command, _("Paste"), "paste")) - .add(MenuItem(MenuItem::Command, _("Emphasize"), "font-emph")); - add(edit); - - Menu documents("documents"); - documents.add(MenuItem(MenuItem::Documents)); - add(documents); - - menubar_.add(MenuItem(MenuItem::Submenu, _("File|F"), "file")) - .add(MenuItem(MenuItem::Submenu, _("Edit|E"), "edit")) - .add(MenuItem(MenuItem::Submenu, - _("Documents|D"), "documents")); - -} - - void MenuBackend::add(Menu const & menu) { menulist_.push_back(menu); @@ -784,28 +884,26 @@ void MenuBackend::add(Menu const & menu) bool MenuBackend::hasMenu(string const & name) const { - return find_if(begin(), end(), - lyx::compare_memfun(&Menu::name, name)) != end(); + return find_if(begin(), end(), MenuNamesEqual(name)) != end(); } Menu const & MenuBackend::getMenu(string const & name) const { - const_iterator cit = find_if(begin(), end(), - lyx::compare_memfun(&Menu::name, name)); + const_iterator cit = find_if(begin(), end(), MenuNamesEqual(name)); if (cit == end()) lyxerr << "No submenu named " << name << endl; - lyx::Assert(cit != end()); + BOOST_ASSERT(cit != end()); return (*cit); } Menu & MenuBackend::getMenu(string const & name) { - MenuList::iterator it = - find_if(menulist_.begin(), menulist_.end(), - lyx::compare_memfun(&Menu::name, name)); - lyx::Assert(it != menulist_.end()); + iterator it = find_if(begin(), end(), MenuNamesEqual(name)); + if (it == end()) + lyxerr << "No submenu named " << name << endl; + BOOST_ASSERT(it != end()); return (*it); }