1490600 - Always use braces for if/for/while statements in XPConnect

Status: RESOLVED FIXED

(Core :: XPConnect, enhancement)

Comment 1

[Jan de Mooij [:jandem]]

The patches I'll post soon are 99% auto-generated and I think can be rubberstamped. (I also checked them myself several times.)

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/loader. r?kmag

Comment 3

[Jan de Mooij [:jandem]]

Attachment: Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 1. r?bholley

Depends on D5651

Comment 4

[Jan de Mooij [:jandem]]

Attachment: Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 2. r?mccr8

Depends on D5652

Comment 5

[Jan de Mooij [:jandem]]

Attachment: Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 3. r?mccr8

Depends on D5653

Comment 6

[Jan de Mooij [:jandem]]

Attachment: Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 4. r?kmag

Depends on D5654

Comment 7

[Jan de Mooij [:jandem]]

Attachment: Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/wrappers. r?bholley

Depends on D5655

Comment 8

[Bobby Holley (:bholley)]

Comment on attachment 9008352 [details]
Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/wrappers. r?bholley

Bobby Holley (:bholley) has approved the revision.

Comment 9

[Bobby Holley (:bholley)]

Comment on attachment 9008348 [details]
Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 1. r?bholley

Bobby Holley (:bholley) has approved the revision.

Comment 10

[Andrew McCreight [:mccr8]]

Comment on attachment 9008349 [details]
Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 2. r?mccr8

Andrew McCreight [:mccr8] has approved the revision.

Comment 11

[Kris Maglione [:kmag]]

Comment on attachment 9008347 [details]
Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/loader. r?kmag

Kris Maglione [:kmag] has approved the revision.

Comment 12

[Andrew McCreight [:mccr8]]

Comment on attachment 9008350 [details]
Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 3. r?mccr8

Andrew McCreight [:mccr8] has approved the revision.

Comment 13

[Kris Maglione [:kmag]]

Comment on attachment 9008351 [details]
Bug 1490600 - Always use braces for if/for/while statements in js/xpconnect/src, part 4. r?kmag

Kris Maglione [:kmag] has approved the revision.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c5a6202e80a0
Always use braces for if/for/while statements in js/xpconnect/loader. r=kmag
https://hg.mozilla.org/integration/autoland/rev/bc86dafc4ae2
Always use braces for if/for/while statements in js/xpconnect/src, part 1. r=bholley
https://hg.mozilla.org/integration/autoland/rev/5fbdba17f904
Always use braces for if/for/while statements in js/xpconnect/src, part 2. r=mccr8
https://hg.mozilla.org/integration/autoland/rev/843c731503cc
Always use braces for if/for/while statements in js/xpconnect/src, part 3. r=mccr8
https://hg.mozilla.org/integration/autoland/rev/3650f0f019d1
Always use braces for if/for/while statements in js/xpconnect/src, part 4. r=kmag
https://hg.mozilla.org/integration/autoland/rev/fa26c3594f36
Always use braces for if/for/while statements in js/xpconnect/wrappers. r=bholley

Comment 15

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/c5a6202e80a0
https://hg.mozilla.org/mozilla-central/rev/bc86dafc4ae2
https://hg.mozilla.org/mozilla-central/rev/5fbdba17f904
https://hg.mozilla.org/mozilla-central/rev/843c731503cc
https://hg.mozilla.org/mozilla-central/rev/3650f0f019d1
https://hg.mozilla.org/mozilla-central/rev/fa26c3594f36