[PATCH] coverity: CID35: fixed wrong condition

Marcus Meissner meissner at suse.de
Tue Feb 6 04:52:07 CST 2007


On Mon, Feb 05, 2007 at 10:29:23PM +0100, Felix Nawothnig wrote:
> Marcus Meissner wrote:
> >-  if(nRelPos>=0) { /* if this or preceding row */
> >+  if(nRelPos<=0) { /* if this or preceding row */
> >     while(nRelPos<=0) {
> 
> Shouldn't that become a "do { ... } while()" then?

No, since there is a return item; after the while ()  loop.

Actually we might want to kill the whole function, it seems not to
be used.

I am just not comfortable with removing it. 

Ciao, Marcus



More information about the wine-devel mailing list