On 12 November 2014 20:40, Emmanuel Bourg <ebo...@apache.org> wrote:
> Le 12/11/2014 21:02, bode...@apache.org a écrit :
>
>> public int readBits(final int count) throws IOException {
>> - if (count < 0 || count > 32) {
>> + if (count < 0 || count > 31) {
>> throw new IllegalArgumentException("count must be between 0 and
>> 32");
>> }
>> while (bitsCachedSize < count) {
>
> I think the message of the exception has to be updated too.
It's OK if between is interpreted as meaning strictly between, i.e.
not equal to the end points.
But I agree that it is not very clear currently and could be improved.
> Emmanuel Bourg
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
