Closed Bug 586830 Opened 9 years ago Closed 9 years ago

Narcissus does not handle empty return statements

Categories

(Other Applications Graveyard :: Narcissus, defect, minor)

Other Branch
x86
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: taustin, Assigned: taustin)

Details

Attachments

(1 file)

$ ./njs -e 'function f() { return; }; f();'
uncaught exception: PANIC: unknown operation undefined: "return"

$ ./js -e 'function f() { return; }; f();'
$
Fixes the parser and interpreter to correctly handle returns with no values.  (Thanks to Shu for his tips on the parser section).  Also, updates narcissus-failures.txt.
Assignee: nobody → taustin
Attachment #473274 - Flags: review?(shu)
Attachment #473274 - Flags: review?(shu) → review+
Changelist: http://hg.mozilla.org/tracemonkey/rev/bc7cd0c9b6d0
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.