Skip to content
This repository was archived by the owner on Aug 4, 2021. It is now read-only.

Remove wrappers#14

Merged
Victorystick merged 1 commit intomasterfrom
remove-wrappers
Dec 2, 2015
Merged

Remove wrappers#14
Victorystick merged 1 commit intomasterfrom
remove-wrappers

Conversation

@Victorystick
Copy link
Copy Markdown
Contributor

@Victorystick Victorystick commented Nov 26, 2015

Remove the CommonJS wrappers

// before ~75 bytes
var a=function(o){o.exports;return o.exports=r.Foo,o.exports}({exports:{}})

// after ~40 bytes
var r={},e={exports:r};e.exports=f.Foo

See #12 (comment) and #13

Fixes #15

@Victorystick Victorystick mentioned this pull request Nov 30, 2015
Closed
@Victorystick
Copy link
Copy Markdown
Contributor Author

Victorystick commented Nov 30, 2015

@Rich-Harris I'd like to get an approval before I merge this and #12.

@Rich-Harris
Copy link
Copy Markdown
Contributor

Rich-Harris commented Dec 1, 2015

👍 – am I right in thinking this supersedes #13?

@Victorystick
Copy link
Copy Markdown
Contributor Author

Victorystick commented Dec 2, 2015

Yup.

Victorystick added a commit that referenced this pull request Dec 2, 2015
@Victorystick
Merge pull request #14 from rollup/remove-wrappers
b2edab6 
Remove wrappers
@Victorystick Victorystick merged commit b2edab6 into master Dec 2, 2015
@Rich-Harris Rich-Harris deleted the remove-wrappers branch December 5, 2015 17:54
@Rich-Harris Rich-Harris mentioned this pull request Dec 5, 2015
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Victorystick @Rich-Harris