[ https://issues.apache.org/jira/browse/TEXT-98?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16099101#comment-16099101 ]
ASF GitHub Bot commented on TEXT-98: ------------------------------------ Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/57 @arunvinudss While I agree that `isDelimiter` should have been private, it is public and was released with commons-text `1.1`. Due to the strict binary compatibilities promise of commons it can not be removed before `2.0`. For now the best we can do is mark it as deprecated and explain that it will be removed in version `2.0`. > Remove isDelimiter() and use HashSets for delimiter check > --------------------------------------------------------- > > Key: TEXT-98 > URL: https://issues.apache.org/jira/browse/TEXT-98 > Project: Commons Text > Issue Type: Improvement > Affects Versions: 1.1 > Reporter: Arun Vinud > Priority: Minor > Fix For: 1.2 > > > The current implementation of *capitalize*, *uncapitalize* and *initials* in > *WordUtils* calls *isDelimiter* for every character and/or codepoint and > isDelimiter loops through the array of delimiters to check for the > occurrence. This is a bit inefficient and results in O(nk) complexity and it > can be reduced to O( n )[if n>k] or O( k ) [if k>n]. -- This message was sent by Atlassian JIRA (v6.4.14#64029)