[Tremor] Bug in Termor bitwise.c

[John Eckerdal]

Hi,

I have been looking into the tremor source code on the lowmem-branch a
bit and there is one section that seems slightly strange:

in the _span() function there is the following section:

    }else{
      /* we've either met the end of decode, or gone past it. halt
	 only if we're past */
      if(b->headend*8<b->headbit)
	/* read has fallen off the end */
	b->headend=-1;
        break;
    }

The comment does not seem to match the comment (and indentation) since
the break; will always be executed if the code goes into this leg of
the if()-else statement.

Or am I misreading the source code?

Thanks in advance,
John Eckerdal