Great minds thinking alike I believe. The alternative is of course to be
rejected!
Your bandaid is identical to the one I posted in a patch yesterday. Not yet
applied to CVS, it would appear. Hint, hint.
Angus
On Tue, 07 Nov 2000, Allan Rae wrote:
> On Tue, 7 Nov 2000, R. Lahaye wrote:
> > "R. Lahaye" wrote:
> > > Hi,
> > >
> > > I've reported this earlier, but may provide slightly more
> > > details here.
> > >
> > > The crash happens as follows:
> > > start LyX
> > > Edit->Preferences->Usage->Spell checker
> > > now keep changing the 'Spell command', without
> > > clicking on any of the <Restore> ... <Close> buttons.
> >
> > One more thing:
> > what is the difference with the same selection-buttons in the
> > Document Layout dialog? These buttons do NOT crash LyX.
>
> There is no feedback callback in Document like there is in Preferences.
> Which is the way I was going to do it but then we wouldn't get feedback
> until you start changing stuff (as I was going to set messages via the
> input() callback).
>
> I've put a bandaid over this segfault in feedback() so you shouldn't be
> able to crash it now.
>
> Angus, see the comments in FormPreferences::feedback().