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

Reply via email to