Wrong position reported for unused destructuring parameters

[nicolo-ribaudo]

This works in JSHint 2.8.0, but not in master.

//jshint esnext: true, unused: true
function fn({ x }) {}

$ bin/jshint test.js
test.js: line 0, col 0, 'x' is defined but never used.
test.js: line 2, col 10, 'fn' is defined but never used.

2 errors

Expected:

test.js: line 2, col 10, 'fn' is defined but never used.
test.js: line 2, col 15, 'x' is defined but never used.

[lukeapage]

It will be the token used in functionparams destructuring i bet. Ive made a pr altering functionparams, will fix this tomorrow on top of that to avoid conflicts. Thanks for finding this.