riched20: Implements FR_MATCHCASE for EM_FINDTEXT

Phil Krylov phil at newstar.rinet.ru
Fri Feb 24 06:39:33 CST 2006


On Fri, 24 Feb 2006 12:24:45 +0100
Alexandre Julliard <julliard at winehq.org> wrote:

> Phil Krylov <phil at newstar.rinet.ru> writes:
> 
> > On Tue, 21 Feb 2006 14:08:38 -0800
> > Lei Zhang <leiz at ucla.edu> wrote:
> >> 
> >> Hi Phil,
> >> 
> >> Ya, it's a typo. Thanks for catching it.
> >
> > Could you kindly fix it then? As this patch was accepted unfixed.
> 
> I accepted the patch because I don't see how pCurItem could be NULL in
> that loop (or rather, it could be NULL, but in that case we'll crash
> before coming back to the loop test). Am I missing something?

Yes, you're right. It can't be NULL because of an earlier test:

para->member.para.nCharOfs + item->member.run.nCharOfs + nEnd - nLen >= nMin

But this is very inevident.

-- Ph.



More information about the wine-devel mailing list