[jira] [Commented] (LANG-915) Wrong locale handling in LocaleUtils.toLocale()

["Benedikt Ritter (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/LANG-915?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13868764#comment-13868764 ] 

Benedikt Ritter commented on LANG-915:
--------------------------------------

Hello Sergio,

I've reviewed your patch. You're using tabs to intend your code. At commons we uses spaces instead, since the representation of the tab character varies between different editors. I've corrected this.

The new method {{StringUtils.occurences(String, String)}} seems to duplicate {{StringUtils.countMatches(CharSequence, CharSequence)}}. I've removed it. Your new method caused {{StringUtilsTest.testStringUtilsCharSequenceContract()}} to fail. It's always good to run {{mvn clean test}} before creating a patch, to be sure everything still works.

All tests pass now. I've committed your patch in rev. 1557378.

Thanks for contributing!

> Wrong locale handling in LocaleUtils.toLocale()
> -----------------------------------------------
>
>                 Key: LANG-915
>                 URL: https://issues.apache.org/jira/browse/LANG-915
>             Project: Commons Lang
>          Issue Type: Bug
>    Affects Versions: 3.1
>            Reporter: Sergio Fernández
>            Priority: Minor
>             Fix For: 3.3
>
>         Attachments: LANG-915-test.patch, LANG-915.patch
>
>
> The static method LocaleUtils.toLocale() fails, at least, to parse 3-char locale strings, which are completelly valid BCP47 locales.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)