[jira] Updated: (LANG-510) Convert StringUtils API to take CharSequence

["Henri Yandell (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/LANG-510?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Henri Yandell updated LANG-510:
-------------------------------

    Comment: was deleted

(was: Alternatively - for now - we could merely call toString on the CharSequence class if it's not a String. 

Basically making the return str.indexOf(...) into:

{code:java}
        if (str instanceof String) {
            return ((String) str).indexOf(searchChar);
        } else {
            return str.toString().indexOf(searchChar);
        }
{code}

Cost to existing String users is an instanceof check and a cast. 

We could then later implement a String indexOf based on charAt and subSequence. Or possibly just charAt if we feel subSequence is expensive.)

> Convert StringUtils API to take CharSequence
> --------------------------------------------
>
>                 Key: LANG-510
>                 URL: https://issues.apache.org/jira/browse/LANG-510
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.*
>            Reporter: Henri Yandell
>             Fix For: 3.0
>
>
> Wherever possible, use CharSequence and not String in the StringUtils API.
> substring can go to subSequence inside code. The general substring method can be replaced with a subSequence method and substring deprecated(?). One question is whether to implement a CharSequence.indexOf type method. Given that it's merely a walking method(?!?), this might be quite handy and would allow a bunch of StringUtils methods to move over.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.