[jira] [Created] (LANG-802) LocaleUtils - unnecessary recursive call in SyncAvoid class

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (LANG-802) LocaleUtils - unnecessary recursive call in SyncAvoid class

Gary D. Gregory (Jira)
Sebb created LANG-802:
-------------------------

             Summary: LocaleUtils - unnecessary recursive call in SyncAvoid class
                 Key: LANG-802
                 URL: https://issues.apache.org/jira/browse/LANG-802
             Project: Commons Lang
          Issue Type: Bug
          Components: lang.*
    Affects Versions: 3.1
            Reporter: Sebb


The SyncAvoid class calls availableLocaleList() which is a method in the containing class that returns SyncAvoid.AVAILABLE_LOCALE_LIST.

This is unnecessarily complex. The code should use the local variable "list" instead.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       
Reply | Threaded
Open this post in threaded view
|

[jira] [Resolved] (LANG-802) LocaleUtils - unnecessary recursive call in SyncAvoid class

Gary D. Gregory (Jira)

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

Sebb resolved LANG-802.
-----------------------

       Resolution: Fixed
    Fix Version/s: 3.2
   

> LocaleUtils - unnecessary recursive call in SyncAvoid class
> -----------------------------------------------------------
>
>                 Key: LANG-802
>                 URL: https://issues.apache.org/jira/browse/LANG-802
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.*
>    Affects Versions: 3.1
>            Reporter: Sebb
>             Fix For: 3.2
>
>
> The SyncAvoid class calls availableLocaleList() which is a method in the containing class that returns SyncAvoid.AVAILABLE_LOCALE_LIST.
> This is unnecessarily complex. The code should use the local variable "list" instead.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira