1450361 - Clean up the style in xpcom/reflect/xptinfo

Status: RESOLVED INCOMPLETE

(Core :: XPCOM, enhancement, P3)

Description

[Andrew McCreight [:mccr8]]

I've run clang-format on this directory, then manually cleaned up bracing of if then elses and fixed the mode lines. I'm going to wait until bug 1438688 lands so I don't bitrot my patches.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1450361, part 1 - Run clang format on xpcom/reflect/xptinfo.

Review commit: https://reviewboard.mozilla.org/r/233310/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233310/

Comment 2

[Andrew McCreight [:mccr8]]

Attachment: Bug 1450361, part 2 - Manually brace ifs in reflect/xptinfo.

Review commit: https://reviewboard.mozilla.org/r/233312/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233312/

Comment 3

[Andrew McCreight [:mccr8]]

Attachment: Bug 1450361, part 3 - Fix mode lines now that the indentation is fixed.

This patch was generated automatically using my modelines.py script.

Review commit: https://reviewboard.mozilla.org/r/233314/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233314/

Comment 4

[Andrew McCreight [:mccr8]]

Nika has a bunch of stuff to change this direct, so I won't break her changes for some random cleanups.

Comment 5

[Andrew McCreight [:mccr8]]

This code was rewritten, so it should be better than it was.