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

Create and map bare-returns into new parse property name #961

Closed
wants to merge 6 commits into from
Closed

Create and map bare-returns into new parse property name #961

wants to merge 6 commits into from

Conversation

Martii
Copy link
Contributor

@Martii Martii commented Feb 6, 2016

Applies to #935

* Following as much previous structure as possible... see analysis at #935 (comment)
* Using `undefined` as default since is technically assigned elsewhere
* Note entirely sure if https://github.com/mishoo/UglifyJS2/blob/master/test/compress/return_undefined.js is the test for this or not
* Change README.md to reflect and add additional sub-note about Userscript IIFE possibility

Applies to #935
@Martii
Copy link
Contributor Author

Martii commented Feb 6, 2016

Claims to have conflicts and I have zero idea why... it's off the mishoo:master that I did all of this (I think). :\

…hoo/UglifyJS2/Issue-935bareReturns

# Conflicts:
#	tools/node.js

Somehow this got foo'd... merging mishoo:master in again... was checked out as master and branch was created off of that master
@Martii
Copy link
Contributor Author

Martii commented Feb 6, 2016

Ahh OUJS fork is out of date... apologies... I can redo this if needed as a new PR so it's not so messsy in commit history. (looks like our fork PR will be the messy one I think)

Martii added 4 commits February 6, 2016 06:59
* The sentence below doesn't make any sense if this isn't done but deviates from the coding pattern

Applies to #935
* Forgot to do this... late night coding woes

Applies to #932
* No idea where this page referenced is store at this time so it might need to be tweaked to the README.md

Applies to #935
* Missed this in the merge conflict on re-paste... apologies for the extra noise g'night :)

Applies to #935
@Martii
Copy link
Contributor Author

Martii commented Feb 6, 2016

Abandoning for #962

@Martii Martii closed this Feb 6, 2016
@Martii Martii deleted the mishoo/UglifyJS2/Issue-935bareReturns branch February 6, 2016 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant