Details

    • Type: Enhancement
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: 9
    • Fix Version/s: 9
    • Component/s: core-libs
    • Labels:
      None
    • Subcomponent:
    • Resolved In Build:
      b114

      Description

      `ChronoField` has four fields for hours - HOUR_OF_DAY, HOUR_OF_AMPM, CLOCK_HOUR_OF_DAY and CLOCK_HOUR_OF_AMPM. The Javadoc for HOUR_OF_DAY should be enhanced to refer to CLOCK_HOUR_OF_DAY and vice versa. The same should be done for HOUR_OF_AMPM and CLOCK_HOUR_OF_AMPM.

      This change will help users find their way between each pair of fields.

      The documentation added to HOUR_OF_DAY will be something like

      "<p>
      See {@link #CLOCK_HOUR_OF_DAY} for the related field that counts hours from 1 to 24.
      "

      It would seem wise to also update the documentation of AMPM_OF_DAY to refer to CLOCK_HOUR_OF_AMPM to ensure users can connect their options.

      This change does not affect the meaning or intent of the spec, it just adds clarification.

      There is also a bug in CLOCK_HOUR_OF_DAY where it refers to "AM/PM" instead of "day".

        Activity

        Hide
        hgupdate HG Updates added a comment -
        URL: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0780b36e8076
        User: ntv
        Date: 2016-04-06 11:46:26 +0000
        Show
        hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0780b36e8076 User: ntv Date: 2016-04-06 11:46:26 +0000
        Hide
        hgupdate HG Updates added a comment -
        URL: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/0780b36e8076
        User: lana
        Date: 2016-04-13 17:56:05 +0000
        Show
        hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/0780b36e8076 User: lana Date: 2016-04-13 17:56:05 +0000

          People

          • Assignee:
            ntv Nadeesh Tv (Inactive)
            Reporter:
            scolebourne Stephen Colebourne
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: