On Fri, Feb 14, 2003 at 09:31:25AM +0100, Andre Poenitz wrote:
> Ok. Let's have a look.
> + // delete empty mathbox (LFUN_BACKSPACE and LFUN_DELETE)
> + bool remove_inset;
> Uninitialized variable. This is actually a bug I just noticed.
> remove_inset may contain a non-zero value which would mean the inset is
> removed in all cases.
This is very bad! :-( I did initialize this value to false. I guess this
mistake happened when I refreshed my tree and typed in relevant changes
all over again and made the patch... I will definitely be more careful
later.
> Spacing:
> Spacing: else {
> Spacing. Use tab for indentation, not spaces.
Sure.
> Andre'
Thank you very much, Andre'.
--
Bo Peng
