I added a patch for CPIO to SANDBOX-294 Cheers On Wed, Mar 18, 2009 at 11:22 AM, sebb <[email protected]> wrote: > On 18/03/2009, Stefan Bodewig <[email protected]> wrote: >> On 2009-03-18, Stefan Bodewig <[email protected]> wrote: >> >> > I'd like to remove some fields, make them private or final and maybe >> > move some methods around before our first release. In particular: >> >> >> I just saw sebb's patch attached to SANDBOX-294 and he seems to agree >> with most of my points. > > Yes - start with no non-private mutable variables (hopefully keep it > that way), and only make methods non-private if they are needed for > the API or the class hierarchy. > >> In addition the patch makes some package >> private fields bzip2 private - which I think is a a good idea as well. >> >> >> Stefan >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
--------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
