You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Peter Lee <pe...@gmail.com> on 2019/11/26 03:46:29 UTC

[Compress]New PRs on github need reviews

Hi all,

I'm Lee and I'm new to commons-compress. I'm a amateur in compression
algorithms. I like commons-compress cause it supports so many
specifications. I'd like to contribute to it.

Recently I pushed 3 PRs in github :
1. Add support for Tar with sparse entries
2. the split/spanned zip support(extracting and constructing)
I believe Stefan is reviewing the PR about split/spanned zip, but it seems
no one is reviewing the other one. I'm hoping if someone could give me some
reviews if time permits, therefore I can make it better. :)

Best regards,
Lee

Re: [Compress]New PRs on github need reviews

Posted by Peter Lee <pe...@gmail.com>.
Thank you Torsten and melloware for your reviews.

The other 2 PRs about extracting and creating split/spanned zip are also
welcome to your reviews if time allows.They contain a lot lines of code and
may take some more time to review. I believe it will be a little difficult
for Stefan to review them alone. Maybe someone can help.

I like commons-compress and I want to keep my code as clear as I can. I
will try my best to answer your reviews ASAP. Hoping my PRs can improve
commons-compress a little bit.

Best regards,
Lee

Torsten Curdt <tc...@vafer.org> 于2019年11月26日周二 下午6:42写道:

> I've added some comments/questions to the PR for Tar.
>
> cheers,
> Torsten
>
> On Tue, Nov 26, 2019 at 4:46 AM Peter Lee <pe...@gmail.com>
> wrote:
>
> > Hi all,
> >
> > I'm Lee and I'm new to commons-compress. I'm a amateur in compression
> > algorithms. I like commons-compress cause it supports so many
> > specifications. I'd like to contribute to it.
> >
> > Recently I pushed 3 PRs in github :
> > 1. Add support for Tar with sparse entries
> > 2. the split/spanned zip support(extracting and constructing)
> > I believe Stefan is reviewing the PR about split/spanned zip, but it
> seems
> > no one is reviewing the other one. I'm hoping if someone could give me
> some
> > reviews if time permits, therefore I can make it better. :)
> >
> > Best regards,
> > Lee
> >
>

Re: [Compress]New PRs on github need reviews

Posted by Torsten Curdt <tc...@vafer.org>.
I've added some comments/questions to the PR for Tar.

cheers,
Torsten

On Tue, Nov 26, 2019 at 4:46 AM Peter Lee <pe...@gmail.com> wrote:

> Hi all,
>
> I'm Lee and I'm new to commons-compress. I'm a amateur in compression
> algorithms. I like commons-compress cause it supports so many
> specifications. I'd like to contribute to it.
>
> Recently I pushed 3 PRs in github :
> 1. Add support for Tar with sparse entries
> 2. the split/spanned zip support(extracting and constructing)
> I believe Stefan is reviewing the PR about split/spanned zip, but it seems
> no one is reviewing the other one. I'm hoping if someone could give me some
> reviews if time permits, therefore I can make it better. :)
>
> Best regards,
> Lee
>