Closed
Bug 1424087
Opened 7 years ago
Closed 6 years ago
Wrong indentation in SandboxFilter.cpp
Categories
(Core :: Security: Process Sandboxing, enhancement, P3)
Tracking
()
RESOLVED
DUPLICATE
of bug 1188202
People
(Reporter: jld, Assigned: jld)
Details
(Whiteboard: sb+)
According to https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style, case labels should be indented by one level from the switch statement, and then non-case-labelled statements should be indented an additional level.
We're doing this wrong, probably because I was just using the Emacs default when I originally wrote the code: the case labels are at the same level as the switch.
This is a minor thing, but now that I've fixed my emacs config it would be nice to not have to overrule it to keep this file consistent with itself.
(This also applies to any other switch statements in security/sandbox/linux, but most of them are small enough that they can just be lazily reindented the next time they're edited.)
Updated•7 years ago
|
Whiteboard: sb+
Assignee | ||
Updated•7 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•6 years ago
|
||
There's a whole-tree reformatting being worked on in bug 1188202 which supersedes this.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•