On Thu, Oct 18, 2007 at 12:57:17PM +0200, Bruno Haible wrote:
> Thanks for reporting all this. I'm applying the appended patch to fix
> all this.

Thanks.

> Colin Watson wrote:
> > Fourthly, gcc complains:
> > 
> >   trim.c: In function ‘trim2’:
> >   trim.c:67: warning: ‘r’ may be used uninitialized in this function
> > 
> > ... and it may be that the state machine in fact renders it impossible
> > for this to be used uninitialised but this wasn't obvious to me.
> 
> I added a comment with the invariant.

Oh, I stupidly misread the 'if (state == 2 && mb_isspace (mbi_cur (i)))'
branch; you're quite right, of course.

-- 
Colin Watson                                       [EMAIL PROTECTED]


Reply via email to