RFR:JDK-8145633-Adjacent value parsing not supported for Localized Patterns
Roger Riggs
Roger.Riggs at Oracle.com
Tue Dec 20 15:18:41 UTC 2016
Hi Nadeesh,
A nice bit of work.
DateTimeFormatterBuilder:
4774: A note about the design to delay the choice of TemporalField and
creating PrinterParser until the locale is known would be
useful
4789: Please add the @param descriptions so the purpose of the new
arguments is clear
4795: Documenting the 5 arg constructor may be useful too
4871: wrap this long line
TCKLocalizedFieldParser.java:
85-87: Single character field names are too cryptic; and could be
final and static?
Thanks, Roger
On 12/20/2016 4:55 AM, nadeesh tv 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.
>
More information about the core-libs-dev
mailing list