Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Impl: RI
    • Labels:
      None

      Description

      After updates to CurrencyUnit, the jdk-stub module no longer compiled.
      Please while it isn't part of the "portable RI", let's always ensure, it's up do date with the spec given this will become the de facto RI for either SE 9, CLDC 8/9 or both.

        Activity

        Hide
        keilw added a comment -

        Fixed the Stub again, a change of the getValid* methods in CurrencyUnit forced it, after code was broken there.
        This module should probably also be included in a CI Job to ensure consistency.

        If returning <code>null</code> is a mandatory requirement for these timestamps, leaving them as Long object instead of a simple long primitive type might be acceptable, but I see a possible overhead or risk of people using AutoBoxing in real life when they gather such primitive values from System.currentTimeMillis(), System.nanoTime() or similar.
        Ultimately that will be up to advise by Oracle Architects, if a complex type is feasible, especially if this affects java.util.Currency, too.

        Show
        keilw added a comment - Fixed the Stub again, a change of the getValid* methods in CurrencyUnit forced it, after code was broken there. This module should probably also be included in a CI Job to ensure consistency. If returning <code>null</code> is a mandatory requirement for these timestamps, leaving them as Long object instead of a simple long primitive type might be acceptable, but I see a possible overhead or risk of people using AutoBoxing in real life when they gather such primitive values from System.currentTimeMillis(), System.nanoTime() or similar. Ultimately that will be up to advise by Oracle Architects, if a complex type is feasible, especially if this affects java.util.Currency, too.

          People

          • Assignee:
            keilw
            Reporter:
            keilw
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved: