From:  Kannan Vijayan <kvijayan@mozilla.com>
Date:  05 Sep 2018 00:31:11 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

I'm a "yes" on this as well.  I really don't care much either way in terms
of personal preference, but the consistency argument works for me as it
does for sfink.

Kannan

On Tue, Sep 4, 2018 at 12:04 PM Steve Fink  wrote:

> 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
>
>
> _______________________________________________
> 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
>