On Mon, Feb 12, 2018 at 12:53 PM, Pascal Schumacher < pascalschumac...@gmx.net> wrote:
> Hello everybody, > > after further consideration I'm not sure if it is a good idea to deprecate > StrMatcher (to convert it to an interface and make the naming match > *StringLookup), as this means that StrBuilder and StrTokenizer also have to > be deprecated. > > Not long ago we told users of these classes to migrate from commons-lang > to commons-text. Now they have to migrate again. Seems a bit much imho. > > What do you think? > What this tells me is that the release of commons-text was made in haste but with the understanding that one of the goals was migration from Commons Lang to Commons Text. Now that we are past that, some sore (IMO) design points are popping up. I just do not think it is a good idea to perpetuate the use of the current/old abstract classes. IMO the design should have been based on interfaces to begin with but, I imagine this code came from Commons Configuration or some such place. So we have another case (probably) where the API was copied in bulk which made migration easier out of wherever into Commons Lang. Now that Commons Text exists and is its own component, it behooves us to shore up any of the shortcoming from this code that eventually landed here and provide a cleaner better API, and interfaces do just that. Gary > Cheers, > > Pascal > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > >