Incorporate uglify's new object property mangling capabilities

[jrhite]

Updated grunt wrapper to be able to hook into uglify's new object property name mangling functionality, the ability to pass files containing property and variable name exceptions and the ability to use a cache to coordinate symbol mangling across multiple calls to uglify.

See uglifyjs v2.4.19 https://www.npmjs.com/package/uglify-js for more info.

[vladikoff]

Thanks for the PR! some tests are failing though :(

[jrhite]

The 4 failing tests have been failing since commit: 8f9bb92. Earlier comments from @XhmikosR say that he is on it and fixing them.

I added 5 new tests specific to the new functionality and they're all passing.

[XhmikosR]

I'm still waiting for a reply from @vladikoff 'cause as you can see from here https://github.com/gruntjs/grunt-contrib-uglify/compare/tests, I only managed to fix 2...

[pdachtera]

+1 for the PR!

[vladikoff]

@jrhite could you please rebase your PR? also no need to update the changelog

[jrhite]

Rebased PR and reverted changelog changes...

Thanks!

On Tue, Mar 31, 2015 at 3:33 AM, Vlad Filippov [email protected]
wrote:

@jrhite https://github.com/jrhite could you please rebase your PR? also
no need to update the changelog

—
Reply to this email directly or view it on GitHub
#312 (comment)
.

[XhmikosR]

This should be reverted too.

[chesserik]

+1 !

[jk-]

really good idea! +1

[yneves]

best idea ever +1

[jrhite]

@XhmikosR I just reverted the package.json version change and rebased to the latest code. Should be good to go.

Cheers!

[XhmikosR]

Needs 2 tailing spaces for a new line for all the newly added types.

[jrhite]

@XhmikosR good eye again! (uggh...MD requires 2 extra spaces like that...still learning it :-) )

I went ahead and add the 2 trailing spaces to all the new options in docs/uglify-options.md. I also fixed some of the older options that were missing this too.

Checked in and rebased. Thanks!

[XhmikosR]

Thanks!

[jrhite]

Thanks!

@XhmikosR when/how will this make it to the npm repo? it will be version 0.8.2, correct?

[XhmikosR]

I don't make the releases myself so when @vladikoff thinks it's OK :P

Though, perhaps we could wait a couple of days in case we merge some of the other PRs too.

[jrhite]

Ah, got it ok. Thanks...

No big rush here, but the sooner the better of course! :-)

On Thu, Apr 2, 2015 at 12:28 PM, XhmikosR [email protected] wrote:

I don't make the releases myself so when @vladikoff
https://github.com/vladikoff thinks it's OK :P

Though, perhaps we could wait a couple of days in case we merge some of
the other PRs too.

—
Reply to this email directly or view it on GitHub
#312 (comment)
.