mpeterv / luacheck / 359

98%

master: 98%

push

travis-ci

Fix event sorting

A warning and an inline option boundary event can share location
when a function declaration is unreachable code (a push event is generated
for each closure start). Previously event comparator assumed that only two
warnings or two inline option events can share location, resulting in an
error when this assumption is violated.

The fix is to compare events of different types by set priority before
falling back to code comparison for warning. Also remove a hack
that partially worked around this issue by adding `code` field
to some inline option events.

Ref #74.

14 of 14 new or added lines in 2 files covered. (100.0%)

3274 of 3344 relevant lines covered (97.91%)

4275.36 hits per line