commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Bodewig <bode...@apache.org>
Subject Re: [VOTE] Release Compress 1.3 based on RC1
Date Wed, 26 Oct 2011 09:17:35 GMT
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-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message