On 2015-01-23, Emmanuel Bourg wrote:

> Le 22/01/2015 17:30, Stefan Bodewig a écrit :

>> Please have a look and identify stuff that looks as if I'd have to
>> reroll a new RC should it come to a vote with the current code base.

> I reviewed the API changes, here are my comments:

> - PasswordRequiredException: the exception is in the sevenz package, do
> we want to move it elsewhere so it can be used later for other formats
> like zip (if that makes sense).

Moved to a new exceptions package.

> - BitInputStream: why not using a long cache instead of an int, like
> BitStream before the refactoring? It might be interesting to do some
> benchmarks and see if it make a difference.

We never needed more than 31 bits, but you are right, I'll try to
experiment a little.

Stefan

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

Reply via email to