On Wed, Feb 04, 2004 at 12:17:48PM +0200, Martin Vermeer wrote:
> > > - virtual void notifyCursorLeaves(idx_type) {}
> > > + virtual void notifyCursorLeaves(idx_type) const {}
> > 
> > NO! The whole idea of these functions is to modify the insets logically.
> > They mustn't be const.
> 
> But why does it even compile then? 

Because you've just changed the signature of the base function which
does not anything, but not e.g.  MathScriptInset::notifyCursorLeaves()
which actually does mutate the thing.

So your patch does not only logically break the system, but also
physically as the 'virtual' machinery won't wokr anymore due to the
signature mismatch.

Andre'

Reply via email to