Closed Bug 1165274 Opened 9 years ago Closed 8 years ago

Code style hinting warns despite disabled by jshint code comment

Categories

(Core :: JavaScript Engine, defect)

40 Branch
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: thomas, Unassigned)

Details

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0
Build ID: 20150513195316

Steps to reproduce:

Run this code from Petka Antonov's Bluebird module:

https://github.com/petkaantonov/bluebird/blob/7caba852c072f2ea3622b40c0b7a6eeb3fae97d3/src/util.js#L177-L186


Actual results:

The console has the warning "SyntaxError: unreachable code after return statement".


Expected results:

Nothing
Component: Untriaged → JavaScript Engine
Product: Firefox → Core
SpiderMonkey's warnings are separate and distinct from jshint's warnings, and we don't pay them any attention.  I don't think we're going to adopt a policy of searching out the warnings they have, that superset the warnings we have (whatever warning this jshint thing is, I'm betting it's not exactly what we have here), and going to extra effort to detect when the developer has opted out of that narrow set of jshint warnings.
Status: UNCONFIRMED → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.