RFR [16/java.xml] 8251561: Fix doclint warnings in the java.xml package

Joe Wang huizhe.wang at oracle.com
Tue Aug 25 00:49:12 UTC 2020


Thanks again Lance, Naoto!  And sorry for yet another review request!  I 
saw Jon's comment on the other doclint thread about replacing @exception 
with @throws. As it happens, we have 172 of them in the sax package as 
well. Please note the changes are all in the org/xml/sax packages:

http://cr.openjdk.java.net/~joehw/jdk16/8251561/webrev_03/

-Joe

On 8/24/20 3:27 PM, Lance Andersen wrote:
> Looks OK Joe
>
>> On Aug 24, 2020, at 5:44 PM, Joe Wang <huizhe.wang at oracle.com 
>> <mailto:huizhe.wang at oracle.com>> wrote:
>>
>> Hi all,  adding Roger's comment for the make file to webrev_02 (the 
>> only change to webrev_01 is Docs.gmk):
>>
>> http://cr.openjdk.java.net/~joehw/jdk16/8251561/webrev_02/
>>
>> Thanks,
>> Joe
>>
>> On 8/21/20 12:49 PM, naoto.sato at oracle.com wrote:
>>> +1
>>>
>>> Naoto
>>>
>>> On 8/21/20 12:24 PM, Lance Andersen wrote:
>>>> Hi Joe,
>>>>
>>>> This looks OK.
>>>>
>>>>
>>>>
>>>>> On Aug 21, 2020, at 2:23 PM, Joe Wang <huizhe.wang at oracle.com> wrote:
>>>>>
>>>>> Pelase review a patch to add the missing @return, @throws, @param 
>>>>> statements in the java.xml package (excluding the DOM component).
>>>>>
>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8251561
>>>>> CSR: https://bugs.openjdk.java.net/browse/JDK-8251995
>>>>>
>>>>> webrev: http://cr.openjdk.java.net/~joehw/jdk16/8251561/webrev_01/
>>>>>
>>>>> Thanks,
>>>>> Joe
>>>>
>>>>
>>>> Best
>>>> Lance
>>>> ------------------
>>>>
>>>>
>>>>
>>>>
>>>> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>>> Oracle Java Engineering
>>>> 1 Network Drive
>>>> Burlington, MA 01803
>>>> Lance.Andersen at oracle.com
>>>>
>>>>
>>>>
>>>>
>>
>
>
> Best
> Lance
> ------------------
>
>
>
> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering
> 1 Network Drive
> Burlington, MA 01803
> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>
>
>
>



More information about the core-libs-dev mailing list