*blush* So I was reading a blog while the test was running!
I just got said conflict, then checked email. Will kill my code and update with yours. Hen On Mon, 14 Jul 2003, Stephen Colebourne wrote: > 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] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
