Harmony
  1. Harmony
  2. HARMONY-140

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

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor 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

        Hide
        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
        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
        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
        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
        Nathan Beyer added a comment -

        Looks good. Thanks.

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

        Verified by Nathan

        Show
        Tim Ellison added a comment - Verified by Nathan

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development