commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Torsten Curdt" <>
Subject Re: [compress] First version of ChangeSet design
Date Fri, 09 Jan 2009 10:28:21 GMT
Applied a slightly changed version. See my comments on the issue.
(Essentially I wasn't a big fan of the worker class in this case.)

The implementation does cover the most simple changes but I guess it
can become more complicated than that. Again see the comments on the

I think we should start writing some testcases for the complex changes
and try to implement them.


On Fri, Jan 9, 2009 at 08:20, Christian Grobmeier <> wrote:
> Hi,
> i made a patch of a first idea of the ChangeSet design which has been
> proposed by Torsten.
> The testcase fails with the ArOutputStream. Problem here is located in
> the Ar-Implementation. Torsten,
> would be quite cool if you can check this out, I didn't see why this
> is happening. Basically it's telling me that
> the offset and the previous entry lenght doesn match (1 byte
> difference). I guessed its simliar to the ArInputStream Bug a fixed
> earlier today, but it isn't. You may see it quickly since you wrote it
> all :-)
> However, I'll gladly accept your review comments on the ChangeSet. I
> tried to keep it as easy as possible, since I don't think there will
> be a lots of new addtions here in the future.
> After the ChangeSet is finished, I will improved the testcases and the
> javadoc. That should do for a first version, does it?
> Cheers,
> Christian
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message