Uploaded image for project: 'Commons Compress'
  1. Commons Compress
  2. COMPRESS-242

X5455_ExtendedTimestamp's API is inconvenient for writers

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5, 1.6
    • Fix Version/s: 1.7
    • Component/s: Archivers
    • Labels:

      Description

      The extra field doesn't do anything useful unless you call setFlags explicitly. This is non-obviuos and not really documented well. The fact that the *_TIME_BIT constants are package private doesn't help either.

      IMHO the constants should be public, the time field setters should set their corresponding flags implicitly when invoked with non-null values (rest when invoked with null values, perhaps) and the documentation clarified.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              bodewig Stefan Bodewig
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: