On 1/7/2014, 3:53 PM, Robert O'Callahan wrote:
I agree that those are the current best practices.

We have a lot of places where we write "void Method() { ... }" all on one
line for trivial setters and getters. I wonder if we should permit that.

I'd rather if we didn't. Often times changing the name/type of something will make that go past the 80/100/whatever line length limit and you'd have to indent it anyways. Plus, consistency FTW.

We have a lot of places where the opening brace of a class declaration is
on the same line as the class name. We should stop doing that. Similar for
function declarations.

Agreed on both.

Cheers,
Ehsan

_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to