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

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

    Details

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

      Description

      This issue is for enhancements to the java.lang.Character class to uplift it to match the Java 5 specification. Changes include the following:

      • New constants
      • valueOf method for boxing - the size of the cache is somewhat arbitrary and can easily be adjusted
      • New Unicode code point methods
      • Test case for all new methods.

      The addition of the code point methods should enable the implementation of the code point methods in StringBuffer and StringBuilder. At least the knowledge gained will help .

      1. Character_java_5_patch.txt
        39 kB
        Nathan Beyer
      2. CharacterTest.java
        20 kB
        Nathan Beyer

        Activity

        Hide
        nbeyer Nathan Beyer added a comment -

        BTW: I ran the test cases against BEA's Java 5 Update 4 R26.0 JRE and Sun's Java 5 Update 5 JRE for reference.

        Show
        nbeyer Nathan Beyer added a comment - BTW: I ran the test cases against BEA's Java 5 Update 4 R26.0 JRE and Sun's Java 5 Update 5 JRE for reference.
        Hide
        tellison Tim Ellison added a comment -

        Thanks Nathan,

        Patch applied to LUNI module java.lang.Character at repo revision 386060.

        Please check that the patch was applied as you expected.

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

        Looks good, thanks.

        Show
        nbeyer Nathan Beyer added a comment - Looks good, thanks.
        Hide
        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