Re: Tom Parker 2007-01-24 <[EMAIL PROTECTED]> > Attached is a patch that appears to fix the bug (by doing some extra > sanity checking). Looks ok so far, but could do with a double-check.
Hi Tom,
thanks for the patch.
> is_empty = i == NONE || st[i].st_size == 0;
> - if ((! is_empty) < p_says_nonexistent[reverse ^ is_empty])
> + if ((i0!=NONE || i!=NONE) && (! is_empty) <
> p_says_nonexistent[reverse ^ is_empty])
> {
> assert (i0 != NONE);
I'm unsure if that's the right thing to do here, the assert must be
there for a reason, so adding the assert condition to the if condition
sounds bad. Then, I didn't check the source closely - does the patch
just cure the symptoms, or the problem?
Christoph
--
[EMAIL PROTECTED] | http://www.df7cb.de/
signature.asc
Description: Digital signature
