https://bz.apache.org/bugzilla/show_bug.cgi?id=62223

Yann Ylavic <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |INVALID
             Status|NEW                         |RESOLVED

--- Comment #4 from Yann Ylavic <[email protected]> ---
(In reply to David Binderman from comment #3)
> 
> Whether or not it is faster, this is good style. Anyone reviewing
> the code will be wondering why the sanity check is after, not before 
> as usual.
Because it's only a stop condition, not a sanity check which is ensured by the
outer "while (*slider < buffer_len)" loop, hence buffer[*slider] is valid
access within the loop.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to