Should unused span cover type annotation as well

[mjbvz]

From microsoft/vscode#50640

TypeScript Version: 3.0.0-dev.201601

Search Terms:

• unused

Code

function foo(x: number) {}

Current behavior:
Only the span for the letter x is marked as unused

Question
Should the unused span cover the type annotation as well

[mhegazy]

Given our previous attempts to mark the keyword as part of the span, i think we are better off just marking the declaration name consistently.. see #24573 for more details.

[typescript-bot]

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.