jshint / jshint / 2908

100%

push

travis-ci

[[FIX]] Do not add cls method names to env. record

A recent refactoring of the class parsing logic [1] removed the function
name inference logic (i.e. integration with the `nameStack` object) in
favor of a similar codepath within the generic `doFunction` routine. By
providing the method name to that routine, the change had an unintended
side effect: method names were used to create new entries in the new
environment record. This was observable from user code in the presence
of JSHint's `unused` linting option: when set, methods would incorrectly
shadow variables that shared the same name.

Continue to rely on the `doFunction` method for name inference (limiting
duplication), but disable the extension of the environment record for
that case.

1 of 1 new or added line in 1 file covered. (100.0%)

4379 of 4447 relevant lines covered (98.47%)

15252.54 hits per line