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...)