<Core Libs Dev>[9] Review request for 8039172: Tidy warnings cleanup for java.net, java.math, java.time, java.rmi
Lance @ Oracle
lance.andersen at oracle.com
Fri Apr 4 12:27:10 UTC 2014
Looks fine
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
Sent from my iPad
On Apr 4, 2014, at 7:53 AM, alexander stepanov <alexander.v.stepanov at oracle.com> wrote:
> 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