[
https://issues.apache.org/jira/browse/LANG-829?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13464362#comment-13464362
]
Sebb commented on LANG-829:
---------------------------
bq. Also, the nameValues entries are only used by the TextStrategy, so could be
saved in the instance.
This won't work, because the Strategy class instances are shared between
formats.
Also, if the same field appears twice in a format, the nameValues approach
would save memory.
However, dropping the KeyValue instances would simplify the code.
> 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
> Priority: Minor
>
> 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
