Skip to content

Commit

Permalink
bugfix mishoo#242
Browse files Browse the repository at this point in the history
  • Loading branch information
colorhook committed Dec 16, 2013
1 parent bd0886a commit fa3300f
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions lib/scope.js
Original file line number Diff line number Diff line change
Expand Up @@ -249,6 +249,11 @@ AST_Scope.DEFMETHOD("next_mangled", function(options){
// we must ensure that the mangled name does not shadow a name
// from some parent scope that is referenced in this or in
// inner scopes.

This comment has been minimized.

Copy link
@rvanvelzen

rvanvelzen Dec 16, 2013

Could you make sure that this comment stays in its appropriate place (that is to say, beneath your addition)?

This comment has been minimized.

Copy link
@colorhook

colorhook Dec 16, 2013

Author Owner

Hi @rvanveizen , do you means the for loop block? what's the better manner, add a new line after my code?

This comment has been minimized.

Copy link
@rvanvelzen

rvanvelzen Dec 16, 2013

The comment above my line comment were about the for loop after the block you inserted. Now it's meaningless.

Since this has been pulled already, I'll just have to harrass @mishoo over this ;)


//bugfix: https://github.com/mishoo/UglifyJS2/issues/242
if(options.except.indexOf(m) !== -1){
continue;
}
for (var i = ext.length; --i >= 0;) {
var sym = ext[i];
var name = sym.mangled_name || (sym.unmangleable(options) && sym.name);
Expand Down

0 comments on commit fa3300f

Please sign in to comment.