JoBoEVD/UglifyJS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix mangling issues with safari

Browse Source 
Fixes #326 and #179

It isn't a really clean solution, but it works. The idea is to
"virtually" reference the function name inside the scope. But to be able
know it isn't *really* referenced, we add a `virtual` flag to
`AST_SymbolRef.reference`.
This commit is contained in:
Richard van Velzen 2013-10-25 15:54:35 +02:00
parent cfd5c6155c
commit 360628de96
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/scope.js
View File

@ -145,6 +145,16 @@ AST_Toplevel.DEFMETHOD("figure_out_scope", function(){
func = node; func = node;
descend(); descend();
func = prev_func; func = prev_func;
if (node instanceof AST_Function && node.name && node.argnames.length) {
var ref = new AST_SymbolRef({
scope: node.argnames[0].scope,
name: node.name.name,
thedef: node.name.thedef
});
ref.reference(true);
}
return true; return true;
} }
if (node instanceof AST_SymbolRef) { if (node instanceof AST_SymbolRef) {
@ -199,9 +209,9 @@ AST_Lambda.DEFMETHOD("init_scope_vars", function(){
this.uses_arguments = false; this.uses_arguments = false;
}); });
AST_SymbolRef.DEFMETHOD("reference", function() { AST_SymbolRef.DEFMETHOD("reference", function(virtual) {
var def = this.definition(); var def = this.definition();
def.references.push(this); if (!virtual) def.references.push(this);
var s = this.scope; var s = this.scope;
while (s) { while (s) {
push_uniq(s.enclosed, def); push_uniq(s.enclosed, def);