On 2011-10-26, Emmanuel Bourg wrote:

> The test coverage of Pack200CompressorInputStream is a bit low, the
> read methods do not seem to be tested.

Only the three-arg version is used, this is true.

I'll look into getting more coverage in trunk so it will improve with
the next release (or the next RC if this vote fails).  I expect to
finish this before the vote closes and will cancel the vote if the
improved coverage shows any issues.

> Also in the matches() method the constant '4' could be replaced by
> CAFE_DOOD.length

Addressed in trunk.

Thanks

        Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to