Johanna Ploog <johanna.pl...@googlemail.com> wrote:
> D" types of rules. My fault. :p) The simplest decision would be to
> either put parentheses around all return values or none, of which I'd
> prefer the former (possibly excepting for function calls) by a wide
> margin.
And I'd prefer the latter (certainly excepting multiline returns) by a
wide margin.
BUT...
> While the above suggestion also follows a simple enough system, if you
> grep "return (true)" vs. "return true", you'll find there are a LOT
> more instances of the former. Surprisingly enough, "return (0)" also
> appears to be more common than "return 0". In the interest of avoiding
> pointless replacements, I vote to keep this system.
I like your logic here, and withdraw any objection. We don't need to
change code. As much as I hate forcing syntax words to look like
function calls (ewww).
Once I've contributed as much code as you I'll complain :-).
> And another point: Many editors allow highlighting of keywords, and a
> lot of the time "return", "true", "false", and "NULL" fall into the
> same category ("reserved keywords" or whatever). Putting brackets
> around values makes it easier to tell at a glance which is which. (And
> yay, I've just realized that *this* is the reason I'm not bothered by
> enums, numbers or actual variables not being bracketed.)
Some editors and colorschemes distinguish -- I like the ones that
color semi-reserved words a darker (or lighter) version of the color
it gives to syntax words.
> Johanna
-Wm
------------------------------------------------------------------------------
What happens now with your Lotus Notes apps - do you make another costly
upgrade, or settle for being marooned without product support? Time to move
off Lotus Notes and onto the cloud with Force.com, apps are easier to build,
use, and manage than apps on traditional platforms. Sign up for the Lotus
Notes Migration Kit to learn more. http://p.sf.net/sfu/salesforce-d2d
_______________________________________________
Crawl-ref-discuss mailing list
Crawl-ref-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/crawl-ref-discuss
