RE: cvs commit: ant/src/main/org/apache/tools/zip ZipOutputStream .java

[Ja...@rzf.fin-nrw.de]

> >        /**
> >   +     * Deflater object for output 
> >   +     *
> >   +     * @since 1.14
> >   +     */
> >   +    protected Deflater def = new
Deflater(Deflater.DEFAULT_COMPRESSION, true);
> >   +    
> >   +    /**
> >   +     * Deflater buffer
> >   +     *
> >   +     * @since 1.14
> >   +     */
> >   +    protected byte[] buf = new byte[512];
> 
> protected and not private (as checkstyle will certainly find out 8-).
> 
> The reason for this is that ZipOutputStream used to extend
> java.util.zip.DeflaterOutputStream which has those two protected
> members.  It now extends FilterOutputStream directly and this will
> provide a certain level of API compatibility.  I'm not sure that we
> really want to go that far.
> 
> Stefan


I think that as a short java comment inside that code would be good. Then
future
committers remind that and don´t modify the accessibility.


Jan

Re: cvs commit: ant/src/main/org/apache/tools/zip ZipOutputStream .java

[Stefan Bodewig <bo...@apache.org>]

On Tue, 22 Jul 2003, Jan Materne <Ja...@rzf.fin-nrw.de> wrote:

> I think that as a short java comment inside that code would be
> good.

True, thanks

        Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org