Harmony
  1. Harmony
  2. HARMONY-160

Three JSE 5 methods are not implemented in java.util.zip.InflaterInputStream

    Details

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

      Description

      Three methods:
      java/util/zip/InflaterInputStream public markSupported()Z,
      java/util/zip/InflaterInputStream public mark(I)V,
      java/util/zip/InflaterInputStream public reset()V,
      are overridden in JSE 5, but they not implemented in Harmony

        Activity

        Hide
        Paulex Yang added a comment -

        Would you pls. to try this patch? thx a lot.

        Show
        Paulex Yang added a comment - Would you pls. to try this patch? thx a lot.
        Hide
        Tim Ellison added a comment -

        Paulex,

        I've taken the methods you sent, but have expanded on the javadoc comments. It would also be good if you could send corresponding test cases for the new implementation (even these trivial methods).

        Fixed in ARCHIVE module java.util.zip.InflaterInputStream at repo revision 382809.

        Thanks.

        Show
        Tim Ellison added a comment - Paulex, I've taken the methods you sent, but have expanded on the javadoc comments. It would also be good if you could send corresponding test cases for the new implementation (even these trivial methods). Fixed in ARCHIVE module java.util.zip.InflaterInputStream at repo revision 382809. Thanks.
        Hide
        Paulex Yang added a comment -

        The prior fix is fine, thx. Here goes the patch for tests.

        Show
        Paulex Yang added a comment - The prior fix is fine, thx. Here goes the patch for tests.
        Hide
        Tim Ellison added a comment -

        Need to check that the tests have been applied.

        Show
        Tim Ellison added a comment - Need to check that the tests have been applied.
        Hide
        Tim Ellison added a comment -

        Thanks Paulex,

        Added a modified version of your test patch to ARCHIVE module.

        Please check that the tests are acceptable to you.

        Show
        Tim Ellison added a comment - Thanks Paulex, Added a modified version of your test patch to ARCHIVE module. Please check that the tests are acceptable to you.
        Hide
        Paulex Yang added a comment -

        looks fine, thank you, Tim

        Show
        Paulex Yang added a comment - looks fine, thank you, Tim
        Hide
        Tim Ellison added a comment -

        Verified by Paulex.

        Show
        Tim Ellison added a comment - Verified by Paulex.

          People

          • Assignee:
            Tim Ellison
            Reporter:
            Paulex Yang
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development