From:  Steve Fink <sfink@mozilla.com>
Date:  05 Sep 2018 00:04:38 Hong Kong Time
Newsgroup:  news.mozilla.org/mozilla.dev.tech.js-engine.internals
Subject:  

Re: Proposal: always use braces for if/for/while statements

NNTP-Posting-Host:  63.245.210.105

Heh. I find the bug argument unconvincing. It's true that the 
always-brace style is slightly less error-prone, but since this is only 
used for very short clauses it seems like a very minor win in practice, 
and much less significant to my eye than the benefits in evening out the 
code density. I find the braceless style much more readable overall -- 
braces add a whole bunch of visual white space that makes it harder to 
parse out the logical groupings of code within a function body.

And yet, I vote "yes", because I find the consistency argument even more 
convincing.

(I'd prefer Gecko to lose the braces, but that's not going to happen.)

I started out always bracing, by the way, but was won over to braceless 
style through exposure to spidermonkey code. At least for 'if' 
statements, anyway -- my personal leaning would be to always brace 
loops. :-/

On 09/04/2018 07:50 AM, David Teller wrote:
> I am all for making braces a requirement, both to unify with Gecko and
> for safety reasons – it would be really annoying to introduce bugs, as
> Apple did not that long ago, just because of missing braces.
>
> Cheers,
>   David
>
> On 04/09/2018 16:41, Jan de Mooij wrote:
>> Hi all,
>>
>> I'd like to propose we change the SpiderMonkey coding style to always brace
>> if/for/while statements.
>>
> _______________________________________________
> dev-tech-js-engine-internals mailing list
> dev-tech-js-engine-internals@lists.mozilla.org
> https://lists.mozilla.org/listinfo/dev-tech-js-engine-internals