José Matos wrote:
On Tuesday 17 July 2007 09:43:12 Abdelrazak Younes wrote:
There's nothing much to think about IMO. The two patches are fixing
_real_ crashes:

Close to the final release any patch that is not a one line makes me nervous, no matter if they fix crashes or not. :-)

I understand but those two are really simple and, this time, I actually made sure there was no "brown paper bag possible. The first one remove some code which is always a good sign :-)


* delete_hideSignal.patch:
1) create a doc
2) insert an ert inset
3) right-click the inset
4) new window
5) delete the ert inset: BOOM!

  There us some kind of poetic justice here. :-)

I am a poet ;-)


* crash_exit_multiview.patch:
With or without multiview, the crash happens randomly if the statusbar
timer times out after the view is closed. Two reason for that:
1) LyXFunc::lyx_view_ is not set to 0
2) quiting is not set to true.

  OK to both.

OK, will commit in a minute, thanks!

Abdel.

Reply via email to