Uploaded image for project: 'Harmony'
  1. Harmony
  2. HARMONY-140

[classlib][luni] java.util.Collections.frequency(Collection,Object) implementation

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Minor
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: None
        • Component/s: Classlib
        • Labels:
          None

          Description

          A patch for the java.util.Collection.frequency(Collection,Object) method. This patch also includes test case updates.

          http://java.sun.com/j2se/1.5.0/docs/api/java/util/Collections.html#frequency(java.util.Collection,%20java.lang.Object)

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            nbeyer Nathan Beyer added a comment -

            Ignore the "Collection_frequency_patch.txt".

            "Collections_new_methods_patch.txt" contains the 'frequency' method as well as the 'reverseOrder(Comparator)' method and the matching test cases.

            I'd change the issue title, but I don't know how or don't have access to do that.

            Spec references:
            http://java.sun.com/j2se/1.5.0/docs/api/java/util/Collections.html#frequency(java.util.Collection,%20java.lang.Object)
            http://java.sun.com/j2se/1.5.0/docs/api/java/util/Collections.html#reverseOrder(java.util.Comparator)

            Show
            nbeyer Nathan Beyer added a comment - Ignore the "Collection_frequency_patch.txt". "Collections_new_methods_patch.txt" contains the 'frequency' method as well as the 'reverseOrder(Comparator)' method and the matching test cases. I'd change the issue title, but I don't know how or don't have access to do that. Spec references: http://java.sun.com/j2se/1.5.0/docs/api/java/util/Collections.html#frequency(java.util.Collection,%20java.lang.Object ) http://java.sun.com/j2se/1.5.0/docs/api/java/util/Collections.html#reverseOrder(java.util.Comparator )
            Hide
            Permalink
            tellison Tim Ellison added a comment -

            Thanks Nathan,

            Patch looks good, applied to LUNI module at repo revision 381974.

            Please check that the patch was applied as you expected.

            Show
            tellison Tim Ellison added a comment - Thanks Nathan, Patch looks good, applied to LUNI module at repo revision 381974. Please check that the patch was applied as you expected.
            Hide
            Permalink
            nbeyer Nathan Beyer added a comment -

            Looks good. Thanks.

            Show
            nbeyer Nathan Beyer added a comment - Looks good. Thanks.
            Hide
            Permalink
            tellison Tim Ellison added a comment -

            Verified by Nathan

            Show
            tellison Tim Ellison added a comment - Verified by Nathan

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development