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' 

Reply via email to