Closed
Bug 809179
Opened 12 years ago
Closed 12 years ago
Please reinstate the js -w and -s command-line options
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: eric.promislow, Unassigned)
Details
Attachments
(1 file, 1 obsolete file)
1.28 KB,
patch
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.0; rv:15.0) Gecko/20100101 Firefox/15.0.1 Build ID: 20120905151427 Steps to reproduce: Komodo uses the js shell to check JavaScript syntax, using a combination of the -C, -w, and -s options for compile-only, give warnings, and strict warnings (not 'use strict'). Actual results: The shell's command-line parser wasn't happy. "-C" is now "-c", and -w and -s are gone. Expected results: Should we look for a different core JavaScript parser to do syntax-checking? We currently offer support for jshint and jslint as alternatives, but both these packages typically lag the state of EcmaScript by a version or two. In a perfect world interfaces never change while internals continue to advance :)
Reporter | ||
Updated•12 years ago
|
Severity: normal → minor
OS: Windows Vista → All
Hardware: x86 → All
Reporter | ||
Comment 1•12 years ago
|
||
Reporter | ||
Updated•12 years ago
|
Attachment #678998 -
Flags: review?
Reporter | ||
Updated•12 years ago
|
Attachment #678998 -
Flags: review? → review?(dvander)
Comment on attachment 678998 [details] [diff] [review] The back-end still implements these options; reinstate them in the front end Review of attachment 678998 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/shell/js.cpp @@ +4764,5 @@ > + reportWarnings = JS_FALSE; > + > + if (op->getBoolOption('s')) { > + JS_ToggleOptions(cx, JSOPTION_STRICT); > + } nit: no braces needed here
Attachment #678998 -
Flags: review?(dvander) → review+
Reporter | ||
Comment 3•12 years ago
|
||
Attachment #678998 -
Attachment is obsolete: true
Reporter | ||
Comment 4•12 years ago
|
||
Komodo app builds and runs correctly with https://bugzilla.mozilla.org/attachment.cgi?id=679911 I don't have checkin status.
Keywords: checkin-needed
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4be5aca0d6cf Thanks for the patch, Eric! One request - to make life easier for those checking in on your behalf, please make sure that your future patches are made per the guidelines below. Thanks again! https://developer.mozilla.org/en-US/docs/Creating_a_patch_that_can_be_checked_in Also, I'm not sure what source repository you generated this patch from, but please make sure futures ones are against mozilla-central as this one most assuredly was not.
Keywords: checkin-needed
Comment 6•12 years ago
|
||
(In reply to Ryan VanderMeulen from comment #5) > I'm not sure what source repository you generated this patch from http://forums.mozillazine.org/viewtopic.php?f=23&t=2557877 suggests aurora.
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4be5aca0d6cf
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Reporter | ||
Comment 8•12 years ago
|
||
Yes, taken from Aurora. I've bookmarked the link for making patches in the future.
You need to log in
before you can comment on or make changes to this bug.
Description
•