Exported var was used before it was defined in 2.9.0

[robrich]

Using JSHint 2.9.0, I get this error: common-lib.js: line 2, col 5, 'commonLib' was used before it was defined. and in previous versions of JSHint I did not. Moving the /* exported ... /* line to the end of the file resolves the symptom. Here is a minimum sample repro project:

.jshintrc:

{
  "latedef": true
}

common-lib.js:

/* exported commonLib */
var commonLib = (function () {
  'use strict';

  return 'the common lib';
}());

command I run:
jshint .

[nicolo-ribaudo]

Possible duplicate of gh-2658

[nicolo-ribaudo]

The regression has been introduced in 08eb4e2

[lukeapage]

@nicolo-ribaudo please comment if you are working on a fix and I won't start working on one..

[lukeapage]

Assuming not..

[lukeapage]

I've started trying to think of the best solution.. the easy thing is to not warn latedef when no usages have tokens.. or have a flag for a non export directive on the usage state.. alternatively don't use usages, but then we have to resolve the export directive to the right scope... Those are my thoughts so far. I am running out of time today though so go ahead if you have started or have an idea..

[lukeapage]

I would probably try the flag route.. so non directive exports set usages.codeUsage or something like that and latedef checks that. but you might have a better idea...

[nicolo-ribaudo]

I have a fix which is almost working... (It just has a bug when exporting variables defined using /* globals) - d5381db

[lukeapage]

go ahead, I will look at the PR.