Closed Bug 560563 Opened 14 years ago Closed 14 years ago

abcasm: getouterscope not supported

Categories

(Tamarin Graveyard :: Tools, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Future

People

(Reporter: cpeyer, Assigned: edwsmith)

Details

Attachments

(3 files)

when trying to use getouterscope with abcasm:

line 142:1 no viable alternative at input 'getouterscope'
line 142:1 Unimplemented opcode "getouterscope"
Flags: flashplayer-qrb?
Target Milestone: --- → Future
Flags: flashplayer-qrb? → flashplayer-qrb+
This generates a folder structure easier for eclipse to grok:

gensrc/grammar/abcasm/*.java
Assignee: nobody → edwsmith
Status: NEW → ASSIGNED
Attachment #440491 - Flags: review?(tharwood)
Comment on attachment 440491 [details] [diff] [review]
Move grammar/abcasm.g3 to grammar/abcasm/abcasm.g3

Just do a diff on those two .g3 files and make sure nothing crept in there.
Attachment #440491 - Flags: review?(tharwood) → review+
I'm adding support for OP_getouterscope, and I'd have to edit this class to keep things consistent.  better to delete the dead code, I reckon.
Attachment #440493 - Flags: review?(tharwood)
Attachment #440493 - Flags: review?(tharwood) → review+
Needed strictequals for the test case, one line change.  looks like support was already there in the rest of abcasm.
Attachment #440498 - Flags: review?(tharwood)
Attachment #440498 - Flags: review?(tharwood) → review+
pushed to tamarin-redux

move grammar and delete Function.java: http://hg.mozilla.org/tamarin-redux/rev/9477ee55a4b4
add getouterscope support: http://hg.mozilla.org/tamarin-redux/rev/9477ee55a4b4
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: