<Core Libs Dev>[9] Review request for 8039172: Tidy warnings cleanup for java.net, java.math, java.time, java.rmi

alexander stepanov alexander.v.stepanov at oracle.com
Fri Apr 4 11:53:53 UTC 2014


Hello Lance,

Thank you for the note; the summaries were removed, please see the 
updated webrev:
http://cr.openjdk.java.net/~yan/8039172/webrev.01/

Regards,
Alexander

On 04.04.2014 15:16, Lance @ Oracle wrote:
> Looks ok but have one concern/question as to why you added the summary 
> attribute to the table tag as it has been deprecated and believe it is 
> not supported in HTML 5.
>
> I would probably not include it
>
> 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 Apr 4, 2014, at 4:55 AM, alexander stepanov 
> <alexander.v.stepanov at oracle.com 
> <mailto:alexander.v.stepanov at oracle.com>> wrote:
>
>> Hello,
>>
>> Could you please review the fix for the following bug:
>> https://bugs.openjdk.java.net/browse/JDK-8039172
>>
>> Webrev corresponding:
>> http://cr.openjdk.java.net/~yan/8039172/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Eyan/8039172/webrev.00/>
>>
>> Just a minor cleanup of javadoc to avoid tidy warnings; no other code 
>> affected.
>>
>> Thanks.
>>
>> Regards,
>> Alexander




More information about the core-libs-dev mailing list