If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Code style hinting warns despite disabled by jshint code comment

RESOLVED WONTFIX

Status

()

Core
JavaScript Engine
RESOLVED WONTFIX
2 years ago
2 years ago

People

(Reporter: Thomas Jensen, Unassigned)

Tracking

40 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
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

Updated

2 years ago
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
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.