commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <>
Subject [jira] [Created] (LANG-884) Simplify FastDateFormat; eliminate boxing
Date Fri, 12 Apr 2013 19:36:15 GMT
Sebb created LANG-884:

             Summary: Simplify FastDateFormat; eliminate boxing
                 Key: LANG-884
             Project: Commons Lang
          Issue Type: Bug
          Components: lang.time.*
    Affects Versions: 3.1
            Reporter: Sebb

FastDateFormat calls the method FormatCache.getDateTimeInstance(Integer, Integer, ...) several
times. There are some issues with this:

- the two Integer parameters could be mixed up
- the int parameters have to be boxed into Integers
- the FastDateFormat class has to 'know' that FormatCache uses null for no date/no time.

The FormatCache class could be extended to add getDate, getTime and getDateTime methods with
int parameters, and the calling sequences would them be a lot more obvious. Instead of:

public static FastDateFormat getDateInstance(final int style) {
    return cache.getDateTimeInstance(style, null, null, null);

one could write:

public static FastDateFormat getDateInstance(final int style) {
    return cache.getDateInstance(style, null, null);

The FormatCache class would then be responsible for boxing the date and time int values as

As well as simplifying the FastDateFormat code, it would also allow the FormatCache code to
be reworked. Only it would know that null is currently used for a missing date or time; and
it could be changed to use a different representation if that was better.

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:

View raw message