You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by "Sebb (JIRA)" <ji...@apache.org> on 2012/09/27 12:36:08 UTC

[jira] [Reopened] (LANG-829) FastDateParser could use Calendar.getDisplayNames for all text fields

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

Sebb reopened LANG-829:
-----------------------

      Assignee: Sebb  (was: Charles Honton)

The change does not address the original issue.
The Map returned from getDisplayNames() is specifically designed to be used for looking up the names in order to return their index values, so we should use that.
                
> FastDateParser could use Calendar.getDisplayNames for all text fields
> ---------------------------------------------------------------------
>
>                 Key: LANG-829
>                 URL: https://issues.apache.org/jira/browse/LANG-829
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.time.*
>            Reporter: Sebb
>            Assignee: Sebb
>            Priority: Minor
>             Fix For: 3.2
>
>
> FastDateParser could use Calendar.getDisplayNames for all text fields.
> This returns a map of names to indexes, which would be easier to search - no need to sort the symbols, and no need for the KeyValue instances.
> Also, the nameValues entries are only used by the TextStrategy, so could be saved in the instance.
> Overall this should reduce memory and be quicker.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira