Closed Bug 1424087 Opened 7 years ago Closed 6 years ago

Wrong indentation in SandboxFilter.cpp

Categories

(Core :: Security: Process Sandboxing, enhancement, P3)

Unspecified
Linux
enhancement

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.)
Whiteboard: sb+
Priority: -- → P3
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.