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

Check whether string actually contains value to prevent from copying the string to reduce memory usage #9329

Merged
merged 1 commit into from Dec 4, 2015

Conversation

mattab
Copy link
Member

@mattab mattab commented Dec 4, 2015

re-created #9328 without submodules updates

@mattab mattab added the c: Performance For when we could improve the performance / speed of Matomo. label Dec 4, 2015
@mattab mattab added this to the 2.15.1 milestone Dec 4, 2015
@mattab mattab added the Needs Review PRs that need a code review label Dec 4, 2015
mattab pushed a commit that referenced this pull request Dec 4, 2015
Check whether string actually contains value to prevent from copying the string to reduce memory usage
@mattab mattab merged commit 2d3b5e5 into master Dec 4, 2015
@mattab mattab deleted the escapelikestring_memory_2 branch December 4, 2015 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Performance For when we could improve the performance / speed of Matomo. Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant