Closed
Bug 672576
Opened 13 years ago
Closed 13 years ago
IonMonkey: some code duplicates functionality of MUseDefIterator
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: rpearl, Assigned: rpearl)
Details
Attachments
(1 file)
3.87 KB,
patch
|
ascheff
:
review+
|
Details | Diff | Splinter Review |
After the refactor from Bug 669984, some portions of the codebase re-implement the isDefinition() checks and toDefinition() casts that are already cleanly implemented by the new iterator. Currently this means that there are no uses of MUseIterator at all, but it probably shouldn't be removed, since there are probably valid uses of it...
Attachment #546839 -
Flags: review?(ascheff)
Comment 1•13 years ago
|
||
Comment on attachment 546839 [details] [diff] [review] replace MUseIterator with MUseDefIterator when appropriate. Review of attachment 546839 [details] [diff] [review]: ----------------------------------------------------------------- Cool
Attachment #546839 -
Flags: review?(ascheff) → review+
Assignee | ||
Comment 2•13 years ago
|
||
http://hg.mozilla.org/users/danderson_mozilla.com/ionmonkey/rev/450add2661fa
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•