RFR [9] 8076224: some tidy warning from core libs
Roger Riggs
Roger.Riggs at Oracle.com
Tue Apr 28 16:55:31 UTC 2015
Yes, looks good to me,
Thanks, Roger
On 4/28/2015 12:09 PM, alexander stepanov wrote:
> Hello Roger,
>
> Thanks!
>
> > Generally, avoiding line breaks of {@link xxxxx} across lines makes
> the source easier to read.
>
> Please see the updated version (no line breaks inside of {@link} tags):
> http://cr.openjdk.java.net/~avstepan/8076224/webrev.01/
> (is that what expected?)
>
> Regards,
> Alexander
>
> On 28.04.2015 17:06, Roger Riggs wrote:
>> Hi Alexander,
>>
>> Thanks for doing this cleanup.
>>
>> Generally, avoiding line breaks of {@link xxxxx} across lines makes
>> the source easier to read.
>> Its is ok to re-wrap lines to make the source more readable.
>>
>> Otherwise, it looks fine to me.
>>
>> Thanks, Roger
>>
>>
>> On 4/28/2015 8:22 AM, alexander stepanov wrote:
>>> Hello,
>>>
>>> Could you please review the following fix
>>> http://cr.openjdk.java.net/~avstepan/8076224/webrev.00/
>>>
>>> for
>>> https://bugs.openjdk.java.net/browse/JDK-8076224
>>>
>>> Just some HTML cleanup for docs (remove tidy warnings; remove some
>>> unnecessary <tt> tags as they are not supported in HTML5).
>>>
>>> Thanks,
>>> Alexander
>>
>
More information about the core-libs-dev
mailing list