Now that it reached my conciousness that BufferView is a view on a buffer
I wonder why there is code like

bool BufferView::Pimpl::dispatch(FuncRequest const & ev)
{  [...]
        switch (ev.action) {
        case LFUN_APPENDIX:
        {
                if (available()) {
                        LyXText * lt = bv_->getLyXText();
                        lt->toggleAppendix(bv_);
                        update(lt,
                               BufferView::SELECT
                               | BufferView::FITCUR
                               | BufferView::CHANGE);
                }
        }
        break;
  [...]


Shouldn't the BufferView be ignorant of the existence of appendices and
rather dispatch the event (preferably in the default branch) to "its"
LyXText?

[Actually, I even think it should go to the buffer first, which in turn
hands it down to some LyXText etc.]

Andre'



-- 
Those who desire to give up Freedom in order to gain Security,
will not have, nor do they deserve, either one. (T. Jefferson)

Reply via email to