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...@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
>
