On Mon, Aug 27, 2007 at 07:36:43PM +0000, Alfredo Braunstein wrote: > > // store some values to be used inside of the handlers > > getPos(beforeDispX_, beforeDispY_); > > > > For trunk, I think we should replace all occurence of > > cur.dispatch() with this->dispatch() in Text3::dispatch(). > > This seems good indeed. In the long run, however, we should think of a way to > enforce this so we don't have to check manually (even some assert at run time > would be nice). Moreover, I don't see what do we gain by using a lyxfunc > internally, probably we should just put the content of the corresponding > dipatch code (that just inserts an inset!) in a standalone function and call > that instead.
Yes. Andre'