On 12 November 2014 20:40, Emmanuel Bourg <[email protected]> wrote: > Le 12/11/2014 21:02, [email protected] 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: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
