-
Notifications
You must be signed in to change notification settings - Fork 39
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 object properties #73
Comments
How would that work? (wrt |
@Nami-Doc: The Google Closure Compiler guys don't allow dynamic member access and actually believe that's an acceptable restriction. Hey, to each his own. |
Yes, and the compiler totally fails at it, giving everybody strange issues. |
There is an error in my exemple sorry :'( There real code is following let u = (function(){
let my_huge_function = function() { return 'test'; };
Object.defineProperties(my_huge_function, {
very_long_name: { value: "hello", writable: false } } );
my_huge_function.other_very_long_name = "hello 2?";
return my_huge_function.other_very_long_name;
})();
alert(u); // print "hello 2?"; and the mangler result : let u=function(){let a=function(){return'test'};return Object.defineProperties(a,{very_long_name:{value:'hello',writable:!1}}),a.other_very_long_name='hello 2?',a.other_very_long_name}();alert(u)
esmangle is a really nice soft for javascript 1.7. |
Do you think disallowing dynamic member access is an acceptable restriction? |
In some cases.. |
Hi there, Thanks for your AMAZING work ;)
In future, do you plan to mangle object properties ?
I use the application from the git repository.
With the following code:
I get:
The text was updated successfully, but these errors were encountered: