RFR: 8037948: Improve documentation for org.w3c.dom package
huizhe wang
huizhe.wang at oracle.com
Mon Jun 30 21:12:57 UTC 2014
On 6/30/2014 1:16 PM, Lance @ Oracle wrote:
> Hi joe,
>
> Should be a comma at the end of line 8 before the and on line 9
comma added.
>
> Looks ok otherwise
I committed the change. Thanks again Lance and Henry!
-Joe
>
> Best
> Lance
>
>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance
> Andersen| Principal Member of Technical Staff | +1.781.442.2037
> <tel:+1.781.442.2037>
> Oracle Java Engineering
> 1 Network Drive <x-apple-data-detectors://34/0>
> Burlington, MA 01803 <x-apple-data-detectors://34/0>
> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
> Sent from my iPad
>
> On Jun 30, 2014, at 4:07 PM, huizhe wang <huizhe.wang at oracle.com
> <mailto:huizhe.wang at oracle.com>> wrote:
>
>> That makes sense. I changed @since back to 1.4 and added L2 in the
>> description:
>>
>> http://cr.openjdk.java.net/~joehw/jdk9/8037948/webrev/
>> <http://cr.openjdk.java.net/%7Ejoehw/jdk9/8037948/webrev/>
>>
>> Thanks,
>> Joe
>>
>> On 6/30/2014 12:35 PM, Henry Jen wrote:
>>> On 06/30/2014 12:26 PM, Lance Andersen wrote:
>>>>
>>>> On Jun 30, 2014, at 3:22 PM, huizhe wang <huizhe.wang at oracle.com
>>>> <mailto:huizhe.wang at oracle.com>>
>>>> wrote:
>>>>
>>>>>
>>>>> On 6/30/2014 12:20 PM, Lance Andersen wrote:
>>>>>> Looks OK Joe, noticed the @since changed as well?
>>>>>
>>>>> Thanks for review.
>>>>>
>>>>> Yes. The support for DOM L3 was since JDK 1.5. Previously this
>>>>> package file mentioned only DOM L2.
>>>>
>>>> OK, thank you. Just seemed strange that the overall version was
>>>> changed as I suspect L3 is compatible with L2. I might have just
>>>> indicated that L3 was added in 1.5 as the support was initially in
>>>> 1.4 I believe. I have no strong preference though
>>>
>>> I agree with Lance here, the package itself is included in JDK since
>>> 1.4.
>>>
>>> In case classes under this package is not properly labeled with
>>> @since tag, they will be considered has the same @since as
>>> package.html, then that would be really misleading.
>>>
>>> Cheers,
>>> Henry
>>
More information about the core-libs-dev
mailing list