On Tue, Nov 04, 2003 at 12:18:45PM +0100, Lars Gullik Bjønnes wrote: > Andre Poenitz <[EMAIL PROTECTED]> writes: > > | On Tue, Nov 04, 2003 at 10:53:53AM +0100, Lars Gullik Bjønnes wrote: > >> Andre Poenitz <[EMAIL PROTECTED]> writes: > >> > >> | I also moved part of the LFUN_INSET_EDIT stuff out of Inset::dispatch > >> | again into edit() functions. Reason is mainly separating two separate > >> | ideas: 1. edit means 'open some dialog' (this is still in dispatch()) > >> | and 2. edt means 'enter some text inset and lock it' (this is back in > >> | edit()) > >> > >> Actually I don't like the dispatch changes at all... should the cursor > >> _ever_ call the BufferView::dispatch? (IMHO no) > > > | Indeed. > > > | And it does not. > > I thought I saw a bv()->dispatch(...) there...
No, jsut inset->dispatch() and finally bv->text->dispatch. DispatchResult LCursor::dispatch(FuncRequest const & cmd) { for (int i = data_.size() - 1; i >= 0; --i) { CursorItem & citem = data_[i]; lyxerr << "trying to dispatch to inset" << citem.inset_ << endl; DispatchResult res = citem.inset_->dispatch(cmd); lyxerr << " result: " << res.val() << endl; switch (res.val()) { [....] default: break; } lyxerr << "# unhandled result: " << res.val() << endl; } lyxerr << "trying to dispatch to main text " << bv_->text << endl; DispatchResult res = bv_->text->dispatch(cmd); lyxerr << " result: " << res.val() << endl; return res; } Andre' -- Those who desire to give up Freedom in order to gain Security, will not have, nor do they deserve, either one. (T. Jefferson or B. Franklin or both...)