Harmony
  1. Harmony
  2. HARMONY-563

Java 5 Enhancement: java.util.Properties should implement methods loadFromXML, storeToXML.

    Details

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

      Description

      XML support is not implemented in java.util.Properties: Java 1.5 methods loadFromXML, storeToXML.

        Activity

        Hide
        Anton Ivanov added a comment -

        Here are patch for java.util.Properties, patch for tests and InvalidPropertiesFormatException class implementation.
        Methods storeToXML(InputStream, String), loadFromXML(InputStream) are implemented, storeToXML(InputStream, String, String) is implemeted but encoding parameter is not supported.

        class java.nio.charset.Charset needs to be implemented in Harmony for support of String encoding parameter of method storeToXML(InputStream, String, String), UTF-8 is used as default encoding and encoding parameter is ignored.

        Show
        Anton Ivanov added a comment - Here are patch for java.util.Properties, patch for tests and InvalidPropertiesFormatException class implementation. Methods storeToXML(InputStream, String), loadFromXML(InputStream) are implemented, storeToXML(InputStream, String, String) is implemeted but encoding parameter is not supported. class java.nio.charset.Charset needs to be implemented in Harmony for support of String encoding parameter of method storeToXML(InputStream, String, String), UTF-8 is used as default encoding and encoding parameter is ignored.
        Hide
        Tim Ellison added a comment -

        Thanks Anton.

        Patch applied to LUNI module at repo revision r412383.

        Looks like this introduces some new dependencies for the LUNI module which I will investigate and update teh manifest accordingly.

        Please check that the patch was applied as you expected.

        Show
        Tim Ellison added a comment - Thanks Anton. Patch applied to LUNI module at repo revision r412383. Looks like this introduces some new dependencies for the LUNI module which I will investigate and update teh manifest accordingly. Please check that the patch was applied as you expected.
        Hide
        Anton Ivanov added a comment -

        Looks good. Thanks Tim.

        Show
        Anton Ivanov added a comment - Looks good. Thanks Tim.
        Hide
        Tim Ellison added a comment -

        Verified by Anton.

        Show
        Tim Ellison added a comment - Verified by Anton.

          People

          • Assignee:
            Tim Ellison
            Reporter:
            Anton Ivanov
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development