Parsing Requires Two Steps

[jshint]

Parsing Requires Two Steps

Agreed, accidentally posted a duplicate, think we can get a better title?

[valueof]

Fixed in master. By default it won't complain but you can roll back to the old behavior by using the new latedef option.

[baryluk]

Can you change default on jshint.com web page? It have option to enable/disable, 'When variable is undefined', but I think having functions declared later is not considered very bad practice, and sometimes unavoidable. I cannot uncheck this box however, because I would like still to check for things declared using var x;, and used before their definition:

function a() {
alert(b);
var b = 5;
}

This is actually subcase of original bug report.

Now unchecking, 'When variable is undefined' option, I do not have false positive "'a' is not defined", but also I do not have true positive "'b' is not defined'.

It makes jshint almost useless (for example for me), if you have lots of forward references in dense code (because filtering out manually functions names from report will take time).

Here is my saved report: http://www.jshint.com/reports/114118
In fact in my case it reports this false "'f' is not defined", despite disabling option about undefined variables.

So my proposition is to split "latedef" option into two separate ones. latedef functions and latedef non-functions.

[baryluk]

As of importance of this issue, let me remember you, that almost similar bug is 3 or 4 times duplicated in issues list! And I still do not consider it fixed.

[valueof]

For simplicity, we don't provide UI for all options on the website, you can always use /*jshint latedef:true */ there though.

[baryluk]

I think there is important distinction beetween, later definitions of variables, and later definitions of functions! They are even working in different ways in language (functions are parsed and added to scope before anyexecution takes place, and can be called, but variables declared with "var" will also be available before declaration place, but will be useless, because will contain "undefined").