Skip to content

Conversation

@ajoslin
Copy link
Contributor

@ajoslin ajoslin commented Mar 21, 2016

When browserify is compiling hundreds of files, this gives a noticeable performance boost.

If you need a benchmark before merging this, I can make one.

When browserify is compiling hundreds of files,
this gives a noticeable performance boost.
@codecov-io
Copy link

Current coverage is 100.00%

Merging #2 into master will not affect coverage as of b1861ae

@@            master      #2   diff @@
======================================
  Files            1       1       
  Stmts           35      36     +1
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit             35      36     +1
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of b1861ae

Powered by Codecov. Updated on successful CI builds.

@ajoslin ajoslin changed the title Add a performant indexOf escape to extract function extract: Add a performant indexOf escape Mar 22, 2016
@ahdinosaur
Copy link
Member

looks good to me, thanks @ajoslin. 😄

@ahdinosaur
Copy link
Member

merged as 7a6e36d, @yoshuawuyts can we has publish?

@yoshuawuyts
Copy link
Contributor

💯 go for it!

@ajoslin
Copy link
Contributor Author

ajoslin commented Mar 25, 2016

Closing in favor of #4

@ajoslin ajoslin closed this Mar 25, 2016
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.

4 participants