Bug 1528751 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

We use to enforce (in reviews) a rule where if a side of an if/elseif/else is braced, all the sides should be braced.
In general I think it makes sense from a readability point of view, and can prevent cases where the else body may be wrongly intepreted; for example bug 1528486.

It would also not prevent people from using non braced constructs, provided all the sides are non braced.
We use to enforce (in reviews) a rule where if a side of an if/elseif/else is braced, all the sides should be braced.
In general I think it makes sense from a readability point of view, and can prevent cases where the else body may be wrongly interpreted; for example bug 1528486.

It would also not prevent people from using non braced constructs, provided all the sides are non braced.

Back to Bug 1528751 Comment 0