Closed Bug 672576 Opened 13 years ago Closed 13 years ago

IonMonkey: some code duplicates functionality of MUseDefIterator

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: rpearl, Assigned: rpearl)

Details

Attachments

(1 file)

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 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+
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.