1) This change should only be supported if it moves methods not yet released.
2) The new utils class must have a clear scope, which will give it a clear name. 3) Size is not the first factor, API is. In this case, the intent of the add/addAll/remove/indexOf methods is to make an array look and behave like a list, hence ArrayAsListUtils. Now, I fear that some of the methods (indexOf?) have already been released, so this fails #1 above. In this case, we might as well just add to the existing class. Stephen --- Gary Gregory <[EMAIL PROTECTED]> wrote: > Good point and I do agree with you. Perhaps we > should postpone any > moving of code around unless we can come up with a > good story for the > users. > > Gary > > > -----Original Message----- > > From: Martin Cooper [mailto:[EMAIL PROTECTED] > > Sent: Tuesday, July 27, 2004 22:10 > > To: Jakarta Commons Developers List > > Subject: Re: [lang] ArrayUtils monster file > > > > We should think about this from the client's > perspective. Clients > > generally don't care how big the underlying > implementation is. What > > they care about is having a meaningful API to > call. If we start > > changing the client API simply because we think > the implementation is > > getting too big, well, then I think we have bigger > problems... ;-) > > > > -- > > Martin Cooper > > > > > > On Tue, 27 Jul 2004 17:20:07 -0400, Gary Gregory > > <[EMAIL PROTECTED]> wrote: > > > Hi there, > > > > > > ArrayUtils is currently a 3800+ line monster > file. I am working on > > > adding "add" and "addAll" methods for 2.1 and am > wondering what > folks > > > would think of a separate class for these ops: > ArrayAddUtils? > > > AddArrayUtils? > > > > > > Thanks, > > > Gary > > > > > > > --------------------------------------------------------------------- > > > 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] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
