Support both -r and -l in Disassemble, allow options in DisassFile

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Paul Biggar, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
Created attachment 489547 [details] [diff] [review]
patch

-r and -l don't both work together, and no options are supported in DisassFile. this fixes both problems.
(Reporter)

Updated

7 years ago
Attachment #489547 - Attachment is patch: true
Attachment #489547 - Attachment mime type: application/octet-stream → text/plain
Attachment #489547 - Flags: review?(igor)

Comment 1

7 years ago
Comment on attachment 489547 [details] [diff] [review]
patch

>+"dis([fun])               Disassemble functions into bytecodes",
>+"disfile('foo.js')        Disassemble script file into bytecodes\n"
>+"  dis() and disfile() can be preceeded with\n"
>+"    \"-r\" (disassemble recursively)\n"
>+"    \"-l\" (show line numbers)",


Nice help!
Attachment #489547 - Flags: review?(igor) → review+
(In reply to comment #1)
> Comment on attachment 489547 [details] [diff] [review]
> patch
> 
> >+"dis([fun])               Disassemble functions into bytecodes",
> >+"disfile('foo.js')        Disassemble script file into bytecodes\n"
> >+"  dis() and disfile() can be preceeded with\n"
> >+"    \"-r\" (disassemble recursively)\n"
> >+"    \"-l\" (show line numbers)",

Should add "arguments " before "can", or something like that. The "()" are a kind of post-fix <code>...</code> convention but not really helping here.

Try this:

"  Both dis and disfile take these options as leading string arguments:\n"

/be

Comment 3

7 years ago
http://hg.mozilla.org/mozilla-central/rev/12d38422c622
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.