jshint / jshint / 1453

100%

Pull #2099

travis-ci

<a href="https://github.com/jshint/jshint/commit/<a class=hub.com/jshint/jshint/commit/2d70b99b0eb1bd0dcfab651edcd37e736ce82078">2d70b99b0<a href="https://github.com/jshint/jshint/commit/2d70b99b0eb1bd0dcfab651edcd37e736ce82078">&quot;&gt;[[FIX]] Remove dead code

The `_.has` utility method checks for the existence of the provided
properties as &amp;quot;own&amp;quot; values of the given object. The removed code
erroneously uses it to check attributes of an Array. The first branch is
(incorrectly) token when statements begin with identifiers/literals &amp;quot;0&amp;quot;,
&amp;quot;1&amp;quot;, or &amp;quot;length&amp;quot;, and the nested branch depends on the block type which
is never a destructuring assignment in this case.

This code has been unreachable since its original introduction into the
codebase [1]. Removing it has no effect on the behavior of JSHint.

[1] commit </a><a class="double-link" href="https://github.com/jshint/jshint/commit/<a class="double-link" href="https://github.com/jshint/jshint/commit/fbf34a576f965aba3affda21e4e4507800162f57">fbf34a576</a>">fbf34a576</a><a href="https://github.com/jshint/jshint/commit/2d70b99b0eb1bd0dcfab651edcd37e736ce82078">
    Author: Guyzmo &lt;guyzmo+github@m0g.net&gt;
    Date:   Wed Mar 6 15:12:15 2013 +0100

    fixed destructuring assign in global context. Updated tests. cf #883

3485 of 3685 relevant lines covered (94.57%)

4071.42 hits per line