commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gary Gregory" <>
Subject RE: [lang] text.Interpolation, on to 2.2
Date Thu, 14 Jul 2005 19:26:18 GMT
Hello VariableFormatters:

I've retro-fitted the VariableResolver interface into the
VariableFormatter class and provided a Map-backed VariableResolver
implementation. I did not change everything to statics. 

I think (and hope) this approach provides an API to address the most
common cases (Map) with the ability to extend functionality by
implementing your own VariableResolver and possibly subclassing. 

If Oliver and others could see how this looks and feels to them I would
appreciate any and all comments; in particular as to the use for


> -----Original Message-----
> From: Oliver Heger []
> Sent: Sunday, July 10, 2005 11:45 AM
> To: Jakarta Commons Developers List
> Subject: Re: [lang] text.Interpolation, on to 2.2
> Gary Gregory wrote:
> >I would like us to reconsider the use of the VariableResolver
> >for VariableFormatter.
> >
> >It seems that this is a cleaner design that does not force
> >or composition as the sole mean of feature extension.
> >
> >Considering the complexity of the StrTokenizer class, I do not think
> >that the earlier concern that VariableFormatter+VariableResolver as
> >framework-like is really valid.
> >
> >The interface VariableResolver could be made to live in the
> >VariableFormatter class we *really* think we need to "hide" this
> >feature.
> >
> >If Oliver is up for it and the list does not say "no, no,
> >I'd like to see a patch to the CVS code that makes VariableFormatter
> >a VariableResolver with an canned implementation for Maps.
> >
> >Gary
> >
> >
> I prefer the VariableResolver approach, too. So if nobody objects, I
> will create a patch, which re-introduces this interface (as an inner
> class) and makes all methods static. (With this approach there is no
> need for creating instances of VariableFormatter, right?)
> Don't know how much time I have in the next days, so this might take
> some days.
> Oliver
> <snip/>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message