RFR:JDK-8145633-Adjacent value parsing not supported for Localized Patterns

Stephen Colebourne scolebourne at joda.org
Tue Dec 20 17:18:13 UTC 2016


In the test provider_adjacentValuePatterns2(), please add

{"YYYYwwe", Y, w, c, "20161201", 2016, 12, 1},

This should succeed, because a single number is all that is needed to
parse day-of-week. (So, it will need to be removed from the invalid
patterns test).

Line 1869 will need to change to "count, count, count" to make the tests pass.

Otehrwise, looks fine, thanks.
Stephen


On 20 December 2016 at 09:55, nadeesh tv <nadeesh.tv at oracle.com> wrote:
> Hi all,
>
> BugId: https://bugs.openjdk.java.net/browse/JDK-8145633
>
> Issue:  Support adjacent value parsing for  Localized Patterns
>
> Webrev:  http://cr.openjdk.java.net/~ntv/8145633/webrev.10/
>
> Pattern 'c' and 'W'  were previously allowed to have 'zero padding' which
> was not explicitly mentioned in CLDR
> (http://unicode.org/reports/tr35/tr35-dates.html).
> To allow 'c' and 'W' to take part in adjacent value  parsing ( at the same
> time, 2 digits are  not required for these patterns), restricted the max
> width of these patterns to 1.
>
> Special thanks to Stephen for the help.
>
> --
> Thanks and Regards,
> Nadeesh TV
>


More information about the core-libs-dev mailing list