<i18n dev> Review Request: JDK-8073211 javadoc of Format parseObject methods should specify NullPointerExceptions
Masayoshi Okutsu
masayoshi.okutsu at oracle.com
Fri Nov 20 14:49:23 UTC 2015
Looks good.
Masayoshi
On 11/20/2015 7:20 PM, Naveen Kumar wrote:
> Hi,
> Please consider this updated webrev.01 for the review of bug
> JDK-8073211
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8073211
> webrev: http://cr.openjdk.java.net/~okutsu/naveen/8073211/webrev.01/
>
> updates in fix:
> Masayoshi pointed out that there was already
> a NullPointerException for pos. So I have merged both of them like this:
>
> * @throws NullPointerException if {@code source} or {@code pos} is null.
>
>
> Sorry for posting it multiple times.
>
>
> regards
> Naveen
>
> On 11/20/2015 3:43 PM, Naveen Kumar wrote:
>> Hi,
>> Please consider this updated webrev.01 for the review of bug
>> JDK-8073211
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8073211
>> webrev: http://cr.openjdk.java.net/~okutsu/naveen/8073211/webrev.01/
>>
>> updates in fix:
>> Masayoshi pointed out that there was
>> already a NullPointerException for pos. So I have merged both of them
>> like this:
>>
>> * @throws NullPointerException if {@code source} or {@code pos} is
>> null. regards Naveen
>>
>>
>>
>> On 11/20/2015 2:28 AM, Naoto Sato wrote:
>>> Looks good to me.
>>>
>>> Naoto
>>>
>>> On 11/19/15 11:31 AM, Naveen Kumar wrote:
>>>> Hi All,
>>>> Please review the following fix in API doc for jdk9
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8073211
>>>> webrev: http://cr.openjdk.java.net/~naoto/naveen/8073211/webrev.00/
>>>>
>>>> Issue:
>>>>
>>>> The 2 parseObject methods in java.text.Format throw a
>>>> NullPointerException if the parameter "source" is null. However,
>>>> this is
>>>> not specified in the javadoc. Please note that this should also be
>>>> checked & corrected in all subclasses of java.text.Format
>>>>
>>>> Fix:
>>>> Documentations of
>>>> /java.text.Format.parseObject(java.lang.String,
>>>> java.text.ParsePosition)/,
>>>> /java.text.Format.parseObject(java.lang.String)/,
>>>> /java.text.MessageFormat.parseObject(java.lang.String,
>>>> java.text.ParsePosition)/,
>>>> /java.text.DateFormat.parseObject(java.lang.String,
>>>> java.text.ParsePosition)/,
>>>> /java.text.NumberFormat.parseObject(java.lang.String,
>>>> java.text.ParsePosition)/ are updated with the following addition:
>>>>
>>>> * @throws NullPointerException if {@code source} is null.
>>>>
>>>>
>>>> regards
>>>> Naveen Kumar
>>
>
More information about the i18n-dev
mailing list