<i18n dev> Request for review - 8021108: Clean up doclint warnings and errors in java.text package
Yuka Kamiya
yuka.kamiya at oracle.com
Fri Jul 26 00:12:37 PDT 2013
Thank you for your reviews, Joe and Masayoshi.
I'll reflect Masayoshi's comment in my fix.
Thanks,
--
Yuka
(2013/07/26 16:03), Masayoshi Okutsu wrote:
> java/text/DateFormat.java:
>
> /**
> * Get a default date/time formatter that uses the SHORT style for both the
> * date and the time.
> + *
> + * @return a date formatter
> */
> public final static DateFormat getInstance() {
>
> This method returns a DateFormat for both date and time. "a date
> formatter" may be confusing. "date/time", which isn't very clear, is
> used in the getDateTimeInstance method. So "a date/time formatter"
> should be fine (for this doclint cleanup).
>
> Otherwise, the fixes look good to me.
>
> Masayoshi
>
> On 7/25/2013 11:42 PM, Yuka Kamiya wrote:
>> Hello,
>>
>> Please review the fix for
>> 8021108: Clean up doclint warnings and errors in java.text package
>> for JDK 8.
>>
>> http://cr.openjdk.java.net/~peytoia/8021108/webrev.00/
>>
>>
>> Joe,
>> Because java.text is not listed in TL team's feature area,
>> ( http://openjdk.java.net/groups/core-libs/ )
>> I don't send this request to core-libs-dev but to you directly.
>> I'd appreciate it if you could do the final check.
>>
>> Thanks,
>> --
>> Yuka
>>
More information about the i18n-dev
mailing list