-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[[FIX]] parse const
declarations in ForIn/Of loops
#2335
Conversation
@@ -4189,8 +4189,8 @@ var JSHINT = (function() { | |||
if (state.tokens.next.id === "var") { | |||
advance("var"); | |||
state.tokens.curr.fud({ prefix: true }); | |||
} else if (state.tokens.next.id === "let") { | |||
advance("let"); | |||
} else if (state.tokens.next.id === "let" || state.tokens.next.id === "const") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: should only be treating let/const specially if esnext
es3 or es5 mode is used
self-reviewed, think it should wait on fixing the behaviour of |
I'm mobile atm so can't compare with #2326 is there any overlap? (Sorry if that seems lazy) |
not that I can see in the diff |
it looks like coveralls.io is having issues |
I don't think it conflicts with my pr :) |
it's not conflicts that's the worry, it's that the behaviour of |
Im working on it - i have work i havent pushed yet as well. If you can
review my pr (im building on top of it so sooner the better for code
review) and get them both in, that would help and i can create my next pr.
If you know of anything const specific (next pr is for latedef and unused
of block scope, so let too) thats very broken and not fixed by this and my
pr, please let me know.
|
Closes #2334