commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Steven Caswell" <st...@mungoknotwise.com>
Subject RE: [lang] StringUtils.ordinalIndexOf() and INDEX_NOT_FOUND
Date Fri, 05 Sep 2003 00:33:01 GMT
Unless you want to be as nerdy as I am and do it so that checkstyle doesn't
complain.


Steven Caswell
steve@mungoknotwise.com
a.k.a Mungo Knotwise of Michel Delving
"One ring to rule them all, one ring to find them..."


> -----Original Message-----
> From: Stephen Colebourne [mailto:scolebourne@btopenworld.com] 
> Sent: Thursday, September 04, 2003 6:22 PM
> To: Jakarta Commons Developers List
> Subject: Re: [lang] StringUtils.ordinalIndexOf() and INDEX_NOT_FOUND 
> 
> 
> Personally I wouldn't use INDEX_NOT_FOUND, as -1 is so well known.
> 
> Stephen
> 
> ----- Original Message -----
> From: "Gary Gregory" <ggregory@seagullsw.com>
> To: "'Jakarta Commons Developers List'" 
> <commons-dev@jakarta.apache.org>
> Sent: Thursday, September 04, 2003 7:22 PM
> Subject: [lang] StringUtils.ordinalIndexOf() and INDEX_NOT_FOUND
> 
> 
> > Hello All,
> >
> > I've added StringUtils.ordinalIndexOf() and introduced a 
> new constant 
> > StringUtils.INDEX_NOT_FOUND = -1. Currently, only 
> ordinalIndexOf uses 
> > this constant.
> >
> > I would like to get a fell if folks think that replacing (where 
> > appropriate), the -1 magic number usage in StringUtils with this 
> > constant
> is
> > a good idea or refactoring gone wild.
> >
> > Thanks,
> > Gary
> >
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 
> 
> 
> 



Mime
View raw message