Closed
Bug 1489242
Opened 7 years ago
Closed 7 years ago
Use new curly-braces-required style in comments, too
Categories
(Core :: JavaScript Engine, enhancement, P3)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: jorendorff, Assigned: jandem)
References
Details
Attachments
(1 file)
There are a handful of comments that omit braces in example code:
https://searchfox.org/mozilla-central/search?q=%5E%5Cs*%5C*+%5Cs*%28if%7Cfor%7Cwhile%29+%5C%28.*%5C%29%5Cs*%24®exp=true&path=js%2Fsrc
Reporter | ||
Updated•7 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•7 years ago
|
||
I didn't change some similar comments that were pseudo code or JS code.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → jdemooij
Status: NEW → ASSIGNED
Pushed by jorendorff@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b60cb229416c
Add {} to example code in comments. r=jorendorff
Comment 3•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•