+1 from me too. I've mailed the Todd privately to see what he thinks [just in case he's not on the list anymore].
As we have the 3 +1's though, I've gone ahead and committed the change. Hen On Sat, 12 Jul 2003, Gary Gregory wrote: > This seems like a good change at the 2.0 level. Obviously, this should be > Javadoc'ed carefully and precisely. > > Gary > > -----Original Message----- > From: Stephen Colebourne [mailto:[EMAIL PROTECTED] > Sent: Saturday, July 12, 2003 02:20 > To: Jakarta Commons Developers List > Subject: [lang] Pre 2.0 - StringUtils.isEmpty(), isNotEmpty() and stringsa > with somespaces > > This got raised on the users list. > > Since this is a major version release, now would be the time to correct this > problem. (ie. for 2.0) > > I propose: > isEmpty() - CHANGE to not trim > isNotEmpty() - no change > isEmptyTrimmed - NEW, does what isEmpty used to > isNotEmptyTrimmed - NEW > > This is an incompatable change for isEmpty(), but seems the best solution, > as we appear to define 'empty' as "" elsewhere and refer to trimming > explicitly. > > Stephen > > ----- Original Message ----- > From: "Todd Jonker" <[EMAIL PROTECTED]> > To: "Jakarta Commons Users List" <[EMAIL PROTECTED]> > Sent: Thursday, June 26, 2003 2:10 AM > Subject: Re: StringUtils.isEmpty(), isNotEmpty() and stringsa with > somespaces > > > > In general, StringUtils seems to munge up the distinctions between null, > > zero-length, and whitespace-only strings. Obviously there is no concrete > > definition of what "empty" means. This makes the whole suite of related > > methods hard to learn, IMO. > > > > Personally, I think "empty" should mean "zero-length or whitespace-only", > > but should not include null. You can't really say that your box is empty > if > > you have no box at all. > > > > Another appropriate term would be "blank". Perhaps a way to fix all this > > would be to deprecate the poorly-defined "empty" methods and replace them > > with well-defined "blank" methods. > > > > .T. > > > > > > On 6/25/03 8:13 AM, [EMAIL PROTECTED] wrote: > > > > > I would tend to agree that it is inconsistent. I can't remember if > > > isNotEmpty() was in release 1.0. If it was then changing it makes me > > > uncomfortable. > > > > > > Stephen > > > > > >> from: =?ISO-8859-1?Q?Reinhard_N=E4gele?= > <[EMAIL PROTECTED]> > > >> date: Wed, 25 Jun 2003 11:02:57 > > >> to: [EMAIL PROTECTED] > > >> subject: Re: StringUtils.isEmpty(), isNotEmpty() and stringsa with > some > > >> spaces > > >> > > >> This behavior is specified in the JavaDocs. However, I also think this > > >> is not really consistent. Any thoughts? > > >> > > >> Reinhard > > >> > > >> > > >> Dmitri Ilyin wrote: > > >> > > >>> Hi, > > >>> > > >>> if is use isEmpty() method of StringUtils with " " (string with some > > >>> spaces) and isNotEmpty() with the same string i get true from both > methods, > > >>> becouse isEmpty trims the parameter string and isNotEmpty doesn't. > > >>> > > >>> I think it is not OK. > > >>> > > >>> regards > > >>> > > >>> Dmitri > > >>> > > >>> > > >>> > > >>> > > >>> --------------------------------------------------------------------- > > >>> 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] > > > > > > > > > > > > > -- > > > > A patriot must always be ready to defend his country against its > > government. -Edward Abbey, naturalist and author (1927-1989) > > > > Defend our country! Vote Howard Dean 2004 > > http://www.DeanForAmerica.com/ > > http://www.MoveOn.org/ > > > > > > --------------------------------------------------------------------- > > 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]
