On Tue, Apr 05, 2011 at 09:45:27AM +0200, Vincent van Ravesteijn wrote: > > >Modified: lyx-devel/trunk/src/frontends/qt4/GuiErrorList.cpp > >============================================================================== > >+ // Don't try to highlight the content of info insets > >+ while (dit.inset().lyxCode() == INFO_CODE) > >+ dit.backwardPos(); > >+ > > It feels wrong that GuiErrorList has to know about specific insets > and that it leads to a crash otherwise. > > Shouldn't this be part of the DocIterator/Cursor class instead ? > Still, it would probably need to ask Inset for some property rather > than introducing a special case for InfoInsets.
I had already tried that, but discovered that asking for editability, for example, was not implemented consistently in all insets. It was some time ago and now I don't remember the details. -- Enrico
