Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8157814

Integrating tzdata2016d causes test/java/text/Format/DateFormat/Bug8081794.java to fail with wrong errorindex

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Not an Issue
    • Icon: P3 P3
    • None
    • 9
    • core-libs
    • None
    • generic
    • generic

      As per the IANA released tzdata2016d integration changes to JDK, few TimeZoneNames started to follow the the new numeric naming conventions for abrreviations(e.g. for TimeZone "Asia/Almaty" the short name became "+06" instead of "ALMT").
      To reflect this changes and fallback to JDKs default format of "GMT[+-]hh:mm", these TimeZoneNames were removed from the resources.

      And this test case failure is caused by removal of time zone names entries from TimeZoneNames*.java files. Only timezone names were removed, but the list of timezones is still stored in tzdb.dat file.
      For removed timezones all zone names(long, short) will be empty strings. When SimpleDateFormat class searches for requested timezone name ("ABC" in our test case) and encounters first zone with empty name, this timezone faulty considered as the requested timezone.
      (regionMatches call with '' and 'ABC' strings: 0 length region match)

      Test failure can be solved by this patch:
      --- a/src/java.base/share/classes/java/text/SimpleDateFormat.java Tue May 24 12:31:30 2016 +0100
      +++ b/src/java.base/share/classes/java/text/SimpleDateFormat.java Wed May 25 01:20:02 2016 +0300
      @@ -1635,8 +1635,8 @@
                   // Checking long and short zones [1 & 2],
                   // and long and short daylight [3 & 4].
                   String zoneName = zoneNames[i];
      - if (text.regionMatches(true, start,
      - zoneName, 0, zoneName.length())) {
      + if (!zoneName.isEmpty() && text.regionMatches(true, start,
      + zoneName, 0, zoneName.length())) {
                       return i;
                   }
               }

      [Thanks to [~aefimov]] for investigating this bug].

            rpatil Ramanand Patil (Inactive)
            rpatil Ramanand Patil (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: