Closed Bug 1511385 Opened 5 years ago Closed 5 years ago

Re-format the stack comment in BytecodeEmitter and helper classes.

Categories

(Core :: JavaScript Engine, enhancement, P3)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla65
Tracking Flags:
Tracking Status
firefox65 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

Re-format the stack comment in BytecodeEmitter and helper classes.
169.27 KB, patch
jorendorff
: review+
Details | Diff | Splinter Review 
Followup for bug 1508178.

As expected, the stack comment alignment breaks due to indent-width change.
Re-running align_stack_comment.py on those files fixes.
Aligned "[stack]" to column 20 (the minimum possible value is 15 right now),
and now all comments fits into 80 chars :)
Attachment #9029384 - Flags: review?(jorendorff)
Attachment #9029384 - Flags: review?(jorendorff) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/49a3abc14837005b7e77b0b280f406af0d06a139
Bug 1511385 - Re-format the stack comment in BytecodeEmitter and helper classes. r=jorendorff
https://hg.mozilla.org/mozilla-central/rev/49a3abc14837
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox65: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.