1
100%
master: 100%

Ran 24 Jul 2018 04:15PM UTC

Files 14

Run time 12min

Badge

Embed ▾

Committed 24 Jul 2018 04:09PM UTC coverage: 98.73%. Remained the same

Job # 2830.1

Build Type

push

travis-ci

Committed by

rwaldron

Commit Message

[[FIX]] Correct restriction on function name

The ECMAScript specification details the following early error for
function-defining productions:

> - If the source code matching this production is strict mode code, it
>   is a Syntax Error if BindingIdentifier is present and the
>   StringValue of BindingIdentifier is "eval" or "arguments".

There were two bugs in the original implementation of this error:

- violations from function expressions with empty bodies would not be
  detected
- violations from function declarations contained within strict mode
  code would be reported twice

Correct the first issue by ensuring that the "(isStrict)" property is
accurately set for functions with empty bodies. Correct the second issue
by re-locating the error detection logic to within the processing logic
for expressions and declarations and inserting an additional condition
within the latter.

(This correction does not require modification to the Test262 whitelist
file, although that test suite does offer coverage for this behavior. At
the revision in use by JSHint, the relevant tests are expressed using
`eval` and therefore unusable by parsers. Those tests have since been
reformatted to correct this; they will be incorporated when JSHint next
updates its reference to Test262.)

Run Details

4043 of 4095 relevant lines covered (98.73%)

3587.5 hits per line

jshint / jshint / 2830 / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2830.1

jshint / jshint / 2830 / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2830.1

jshint / jshint / 2830 / 1
100%
master: 100%

README BADGES
x