Skip to content
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

Mangle - Fix for classes and program scope #140

Merged
merged 6 commits into from
Sep 12, 2016
Merged

Conversation

boopathi
Copy link
Member

@boopathi boopathi commented Sep 2, 2016

EDIT: DO NOT MERGE.. yet

Still doesn't cover some cases. #138

EDIT 1: Updated.

+ programPath.traverse is never going to send Program type inside the traverse. A class declaration is bound to the class scope as well as the program scope - so class declarations in global scope is mangled.
@boopathi boopathi changed the title Fix program scope Mangle - Fix program scope Sep 2, 2016
@boopathi boopathi changed the title Mangle - Fix program scope Mangle - Fix for classes and program scope Sep 10, 2016
@boopathi
Copy link
Member Author

Updated!.

@kangax kangax merged commit d8c7ebe into master Sep 12, 2016
@kangax
Copy link
Member

kangax commented Sep 12, 2016

Looks good, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants