Closed
Bug 1383155
Opened 7 years ago
Closed 7 years ago
remove redundant SrcNoteType casts
Categories
(Core :: JavaScript Engine, enhancement)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: tromey, Assigned: tromey)
Details
Attachments
(1 file)
While working on bug 1370648 I noticed code like: SrcNoteType type = (SrcNoteType) SN_TYPE(sn); However, the cast here is redundant, because SN_TYPE does it: https://dxr.mozilla.org/mozilla-central/rev/7d2e89fb92331d7e4296391213c1e63db628e046/js/src/frontend/SourceNotes.h#113 I think the redundant casts should be removed.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8888881 [details] Bug 1383155 - remove redundant casts of SN_TYPE; https://reviewboard.mozilla.org/r/159912/#review165284 Easy!
Attachment #8888881 -
Flags: review?(jimb) → review+
Pushed by ttromey@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/40b0b14356e1 remove redundant casts of SN_TYPE; r=jimb
Comment 4•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/40b0b14356e1
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in
before you can comment on or make changes to this bug.
Description
•