Harmony
  1. Harmony
  2. HARMONY-239

[classlib][luni] Java 5 enhancements for java.lang.Float

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Classlib
    • Labels:
      None
    • Estimated Complexity:
      Moderate

      Description

      Patch for Java 5 enhancements to java.lang.Float.

      1. Float_java_5_patch.diff
        11 kB
        Nathan Beyer
      2. Float_java_5_no_cache.diff
        10 kB
        Nathan Beyer

        Activity

        Nathan Beyer created issue -
        Nathan Beyer made changes -
        Field Original Value New Value
        Attachment Float_java_5_patch.diff [ 12324474 ]
        Hide
        Nathan Beyer added a comment -

        After some thought, the caching that's implemented in the valueOf method of the attached diff may not be appropriate or valuable, so feel free to remove it.

        Show
        Nathan Beyer added a comment - After some thought, the caching that's implemented in the valueOf method of the attached diff may not be appropriate or valuable, so feel free to remove it.
        Tim Ellison made changes -
        Assignee Tim Ellison [ tellison ]
        Hide
        Nathan Beyer added a comment -

        There was an issue with the cache that I put into the valueOf method, so I've removed it. The 'no cache' diff should reflect this, both in the class and test case.

        Show
        Nathan Beyer added a comment - There was an issue with the cache that I put into the valueOf method, so I've removed it. The 'no cache' diff should reflect this, both in the class and test case.
        Nathan Beyer made changes -
        Attachment Float_java_5_no_cache.diff [ 12324747 ]
        Hide
        Tim Ellison added a comment -

        Thanks Nathan,

        The no cache patch looks better.
        Applied to LUNI module java.lang.Float at repo revision 390180.

        Please check that the patch was applied as you expected.

        Show
        Tim Ellison added a comment - Thanks Nathan, The no cache patch looks better. Applied to LUNI module java.lang.Float at repo revision 390180. Please check that the patch was applied as you expected.
        Tim Ellison made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Hide
        Nathan Beyer added a comment -

        Looks good, thank you.

        Show
        Nathan Beyer added a comment - Looks good, thank you.
        Hide
        Tim Ellison added a comment -

        Verified by Nathan

        Show
        Tim Ellison added a comment - Verified by Nathan
        Tim Ellison made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        7d 11h 22m 1 Tim Ellison 31/Mar/06 03:08
        Resolved Resolved Closed Closed
        15h 48m 1 Tim Ellison 31/Mar/06 18:57

          People

          • Assignee:
            Tim Ellison
            Reporter:
            Nathan Beyer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development