[
https://issues.apache.org/jira/browse/TEXT-98?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16099520#comment-16099520
]
ASF GitHub Bot commented on TEXT-98:
------------------------------------
Github user ameyjadiye commented on the issue:
https://github.com/apache/commons-text/pull/57
@arunvinudss , just addition to @PascalSchumacher comment , at this point
we don't know other than Commons text who else having dependancy on
`isDelimiter` so better we can make it depricated and we can remove code all
together in 2.x
> 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)
