On Wed, May 4, 2011 at 10:21 AM, Matt Benson <gudnabr...@gmail.com> wrote:

> On Wed, May 4, 2011 at 8:14 AM, Gary Gregory <garydgreg...@gmail.com>
> wrote:
> > On May 4, 2011, at 8:36, Matt Benson <gudnabr...@gmail.com> wrote:
> >
> >> On Wed, May 4, 2011 at 5:39 AM, Stephen Colebourne <
> scolebou...@joda.org> wrote:
> >>> (review now back from holiday)
> >>>
> >>> -1
> >> [SNIP]
> >>> I'm still looking for ImmutablePair to be final too, but perhaps I
> >>> missed the conclusion of that debate... (NOT DONE in SVN)
> >>>
> >>
> >> Gary, now that you have toString(format) back, can you function with
> this?
> >
> > Yes. I do not need that class to be Formattable.
>
> The question was whether you need it to be extensible.  ;)
>

Ah, well, it would make it easier to port my current code: create subclasses
with the desired formatting and replace package names. Each subclass would
call toString(format) and that's it. The alternative is to either call
toString(format) from the call sites (sure, I could refactor that in a Utils
class) or have my factory classes create pair wrappers. But it seems easier
to keep reusing the current code than creating wrappers. If the class is
final, I have to go the Utils route, I do not have the choice.

I just do not see the point of hand-cuffing users by making the class final.
I can always find new ways of shooting myself in the foot... ;)

Gary


> Matt
>
> >
> > Gary
> >
> >>
> >> Matt
> >>
> >>> I also made some minor javadoc changes.
> >>>
> >>> Stephen
> >>>
> >>>
> >>>
> >>> On 29 April 2011 07:47, Henri Yandell <flame...@gmail.com> wrote:
> >>>> Lang is ready to consider 3.0 release again.
> >>>>
> >>>> RC3 is available here:
> >>>>
> >>>>  http://people.apache.org/~bayard/commons-lang3-3.0-RC3/
> >>>>
> >>>> Maven artifacts:
> >>>>
> >>>>  http://people.apache.org/~bayard/commons-lang3-3.0-RC3/maven/
> >>>>
> >>>> Website:
> >>>>
> >>>>  http://people.apache.org/~bayard/commons-lang3-3.0-RC3/site/
> >>>>
> >>>> Note that there is a 2.6->3.0 Clirr report in the site that may prove
> useful:
> >>>>
> >>>>
> http://people.apache.org/~bayard/commons-lang3-3.0-RC3/site/lang2-lang3-clirr--report.html
> >>>>
> >>>> This vote will close no sooner than in 72 hours time, 0700 GMT 2-May
> 2011.
> >>>>
> >>>> ================
> >>>>   [ ] +1
> >>>>   [ ] -1, with reason
> >>>> ================
> >>>>
> >>>> Hen
> >>>>
> >>>> ---------------------------------------------------------------------
> >>>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> >>>> For additional commands, e-mail: dev-h...@commons.apache.org
> >>>>
> >>>>
> >>>
> >>> ---------------------------------------------------------------------
> >>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> >>> For additional commands, e-mail: dev-h...@commons.apache.org
> >>>
> >>>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> >> For additional commands, e-mail: dev-h...@commons.apache.org
> >>
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > For additional commands, e-mail: dev-h...@commons.apache.org
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>


-- 
Thank you,
Gary

http://garygregory.wordpress.com/
http://garygregory.com/
http://people.apache.org/~ggregory/
http://twitter.com/GaryGregory

Reply via email to