Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Eval error when eval method exists in a object #738

Closed
arunoda opened this Issue · 2 comments

4 participants

@arunoda

Following line cause an error and It is a legitimate and It has no relationship to eval.

result =  schemaEvaluator.eval(experimentId, schema);

Error is shown here: http://arunoda.me/blog/jshint-evil-eval-check.html

@rwaldron
Owner

This should've been corrected for ES5

@mbildner mbildner referenced this issue from a commit in mbildner/jshint
@mbildner mbildner [[FIX]] Permit "eval" as object key
Makes it possible to use "eval" as a key in an object. Previously this
would have emitted a "W061" warning. "eval" used as the key for a
presumed global ("window" in "browser:true" and "global" in
"node:true") will still emit the warning.

Fixes jshint#738
44d5838
@mbildner mbildner referenced this issue from a commit in mbildner/jshint
@mbildner mbildner [[FIX]] Permit "eval" as object key
Makes it possible to use "eval" as a key in an object. Previously this
would have emitted a "W061" warning. "eval" used as the key for a
presumed global ("window" in "browser:true" and "global" in
"node:true") will still emit the warning.

Fixes jshint#738

Adds unit tests for permitting using "eval" as object key
5b6f804
@mbildner

this pr should fix that #2188

@mbildner mbildner referenced this issue from a commit in mbildner/jshint
@mbildner mbildner [[FIX]] Permit "eval" as object key
Makes it possible to use "eval" as a key in an object. Previously this
would have emitted a "W061" warning. "eval" used as the key for a
presumed global ("window" in "browser:true" and "global" in
"node:true") will still emit the warning.

Fixes jshint#738

Adds unit tests for permitting using "eval" as object key
b5f5d5d
@rwaldron rwaldron closed this in #2188
@jugglinmike jugglinmike referenced this issue from a commit in jugglinmike/jshint
@mbildner mbildner [[FIX]] Permit "eval" as object key
Makes it possible to use "eval" as a key in an object. Previously this
would have emitted a "W061" warning. "eval" used as the key for a
presumed global ("window" in "browser:true" and "global" in
"node:true") will still emit the warning.

Fixes jshint#738

Adds unit tests for permitting using "eval" as object key
d2e9546
@jugglinmike jugglinmike referenced this issue from a commit in jugglinmike/jshint
@mbildner mbildner [[FIX]] Permit "eval" as object key
Makes it possible to use "eval" as a key in an object. Previously this
would have emitted a "W061" warning. "eval" used as the key for a
presumed global ("window" in "browser:true" and "global" in
"node:true") will still emit the warning.

Fixes jshint#738

Adds unit tests for permitting using "eval" as object key
06a0c34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.