For me this is not too terribly religious, as opposed to say, oh, something
like bracket placement ;)
Either way is fine with me.
On Apr 12, 2005 3:13 AM, Stephen Colebourne <[EMAIL PROTECTED]>
wrote:
>
> Personally I think its a lot more readable with the brackets (unless its
> a simple return). Thus I tend to code as:
>
> return value;
> return method();
> return (expression);
>
> But if others dislike this, I'll try to remembre not to do it on lang.
> Better put it in the lang guidelines though.
>
> Stephe
>
> Henri Yandell wrote:
> > -1 to return (foo) :)
> >
> > You get my vote to commit. I'm pretty sure the general agreement is to
> > not have brackets on return statements, but I'm unsure if it's some
> > contributors or some committers who like the brackets.
> >
> > Hen
> >
> > On Apr 11, 2005 6:56 PM, Gary Gregory <[EMAIL PROTECTED]>
> wrote:
> >
> >>Hello:
> >>
> >>I've committed some small clean ups in the text package which will not
> >>be shipped in 2.1.
> >>
> >>What I found while doing this is a code style inconsistency that I've
> >>fixed locally but not committed yet. Since this involves style (religion
> >>;-) I did not want to commit without posting here first:
> >>
> >>Throughout [lang], some return statements are coded as the "standard":
> >>
> >> return expression;
> >>
> >>While others are coded as with extra C-style "(" and ")":
> >>
> >> return (expression);
> >>
> >>I did not find any pattern to either style. I removed extra "(" and ")"
> >>in return expressions on my local store.
> >>
> >>Is this a religious issue for some [lang]'ers out there or should I just
> >>commit?
> >>
> >>Thanks,
> >>Gary
> >>
> >>-----Original Message-----
> >>From: Henri Yandell [mailto:[EMAIL PROTECTED]
> >>Sent: Sunday, April 10, 2005 5:54 PM
> >>To: Jakarta Commons Developers List
> >>Subject: [lang] VOTE 2.1 release
> >>
> >>Proposing a vote to go ahead and release Commons Lang 2.1.
> >>
> >>The release will pretty much match the release-candidate that is in:
> >>http://www.apache.org/~bayard/commons-lang-2.1/
> >>
> >>Stephen's recent defaultIfEmpty method will be included in the release.
> >>
> >>If the vote is successful, I'll target spending next weekend
> >>performing the release (unless anyone wants to volunteer).
> >>
> >>[ ] +1
> >>[ ] -1
> >>
> >>Hen
> >>
> >>---------------------------------------------------------------------
> >>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]
>
>
--
Steven Caswell
[EMAIL PROTECTED]
Take back the web - http://www.mozilla.org
