No, you haven't committed this change, I have.... Did neither of us get a conflict? Stephen
----- Original Message ----- From: "Henri Yandell" <[EMAIL PROTECTED]> To: "Jakarta Commons Developers List" <[EMAIL PROTECTED]> Sent: Monday, July 14, 2003 11:37 PM Subject: RE: [lang] Pre 2.0 - StringUtils.isEmpty(), isNotEmpty() and stri ngsa with somespaces > > +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] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]