<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