option relax "wrap the /regexp/ literal in parenthesis" warning #1218
Comments
benesch
added a commit
to benesch/jshint
that referenced
this issue
Dec 20, 2013
This warning flags any bare regexp after a return or yield statment as
ambiguous. (Example: `return /[0-9]/`.) The unnecessary fix is to wrap
the regexp in parens like so:
return (/[0-9]/)
This is a useless warning. An unwrapped regexp following a return
statement is not ambiguous to humans or parsers. The division operator
(/) must follow an expression; a regexp literal must *not* follow an
expression. This warning flags regexp literals that appear immediately
after a return statement and thus never after an expression.
No reason to optionalize this warning, as it's not useful.
References:
http://groups.yahoo.com/neo/groups/jslint_com/conversations/topics/345
eslint/eslint#324
jshint#245
jshint#1218
|
Fixed in #1433 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
/[0-9]/.exec('123')throwing a warning is pretty annoying.The text was updated successfully, but these errors were encountered: