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

Dumper::encodeString - use mb_substr #222

Merged
merged 1 commit into from Sep 12, 2016
Merged

Dumper::encodeString - use mb_substr #222

merged 1 commit into from Sep 12, 2016

Conversation

@matej21
Copy link
Contributor

matej21 commented Sep 12, 2016

iconv_substr is very f*cking slow. Even the do-while fallback is few thousand times faster :D

@dg dg merged commit a8fe81f into nette:master Sep 12, 2016
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@matej21 matej21 deleted the matej21:mb_substr branch Sep 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.