commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stefan Bodewig (JIRA)" <>
Subject [jira] Commented: (SANDBOX-294) The field is never read locally
Date Fri, 13 Mar 2009 05:35:50 GMT


Stefan Bodewig commented on SANDBOX-294:

I'd rather turn this in a more general "think about protected API" task that applies to some
things in the zip package as well.

As the checkstyle comments indicate, these fields are protected because they used to be that
way in early versions of Ant - this is a backwards compatibility requirement compress doesn't
share, so we are free to change things that don't really make sense (we don't want subclasses
to change the buffer, for example).

> The field is never read locally
> --------------------------------------------------------
>                 Key: SANDBOX-294
>                 URL:
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: Compress
>            Reporter: Sebb
>            Priority: Minor
> The field is never read locally.
> Is there any need for this field, which is stored by the constructor and also passed
to the TarBuffer constructor?
> Also, the fields "buffer" and "oneBuf" could be made final.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message