Closed
Bug 539071
Opened 15 years ago
Closed 15 years ago
Two calls to isMachineCompatible are redundant
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tamarin Graveyard
Virtual Machine
Tracking
(Not tracked)
VERIFIED
FIXED
flash10.1
People
(Reporter: edwsmith, Unassigned)
Details
Attachments
(1 obsolete file)
in Verifier.cpp, canAssign() calls isMachineCompatible() internally, so the expression
(canAssign(x,y) && isMachineCompatible(x,y))
can be simplified to
(canAssign(x,y))
|
Reporter | |
Updated•15 years ago
|
Attachment #421124 -
Attachment is patch: true
Attachment #421124 -
Attachment mime type: application/octet-stream → text/plain
Attachment #421124 -
Flags: review?(tharwood)
|
||
Updated•15 years ago
|
Attachment #421124 -
Flags: review?(tharwood) → review+
|
|
Reporter | |
Comment 1•15 years ago
|
||
Comment on attachment 421124 [details] [diff] [review]
Removes redundant check
pushed http://hg.mozilla.org/tamarin-redux/rev/f01535929670
Attachment #421124 -
Attachment is obsolete: true
|
|
Reporter | |
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•