https://sourceware.org/bugzilla/show_bug.cgi?id=30354

Nick Clifton <nickc at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14851|0                           |1
        is obsolete|                            |

--- Comment #7 from Nick Clifton <nickc at redhat dot com> ---
Created attachment 14860
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14860&action=edit
Proposed patch

Hi Torbjörn

> Just looking at the patch, I'm a bit confused about the variable 
> "extra_marks_added". > The initial value is true, and in the function body, 
> it's set to true in certain 
> cases, but it's never set to false and thus, the if-statement at the end of 
> the patch 
> should always happen. Is this the desired behavior?

Oops - snafu.  Please try out this corrected version which initialises
extra_marks_added to false...

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Reply via email to