Closed
Bug 1489242
Opened Last year
Closed Last year
Use new curly-braces-required style in comments, too
Categories
(Core :: JavaScript Engine, enhancement, P3)
Core
JavaScript Engine
P3
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•Last year
|
Priority: -- → P3
|
Assignee | |
Comment 1•Last year
|
||
I didn't change some similar comments that were pseudo code or JS code.
|
|
Assignee | |
Updated•Last year
|
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•Last year
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/b60cb229416c
Status: ASSIGNED → RESOLVED
Closed: Last year
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
•