This repository has been archived by the owner. It is now read-only.

Removed JString usage in JStringNormalise #1162

Merged
merged 2 commits into from Apr 20, 2012

Conversation

Projects
None yet
3 participants
Contributor

eddieajau commented Apr 19, 2012

Using JString instead of normal string methods causes a significant performance hit. Removing the JString wrapper fixes this problem. As this class was designed for programatic values, this should not cause much of an issue.

Also deprecate JString::splitCamelCase and merged it with JStringNormalise::fromCamelCase.

eddieajau and others added some commits Apr 19, 2012

Add ASCII notes to JStringNormalise class methods.
Deprecate JString::splitCamelCase (merged with JStringNormalise::fromCamelCase).

ianmacl added a commit that referenced this pull request Apr 20, 2012

Merge pull request #1162 from eddieajau/robschley-normalise
Removed JString usage in JStringNormalise

@ianmacl ianmacl merged commit 6888cf1 into joomla:staging Apr 20, 2012

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.