RFR [9] 8081517: minor cleanup for docs

alexander stepanov alexander.v.stepanov at oracle.com
Fri Jun 5 14:59:40 UTC 2015


 > indentation is in the scope of your cleanup

No, I didn't check the indents, lengths of lines etc. (sorry). The 
intention was just to fix some tidy warnings (like empty <p> tags) and 
javadoc errors like invalid parameter names, invalid tags etc. Tags  
<code></code> were replaced with {@code} in couple of files just because 
of some connected warnings like unclosed tags etc.

Regards,
Alexander

On 05.06.2015 17:12, Lance Andersen wrote:
> Hi Roger
> On Jun 5, 2015, at 10:08 AM, Roger Riggs <Roger.Riggs at oracle.com> wrote:
>
>> 2)
>> If indentation is in the scope of your cleanup…
> Agree with you and I did not comment on this as I felt it this should be done separately as there is a fair amount that can be done in these classes for clean up.
>
> Best
> lance
>>
>> In JdbcRowSetImpl.java, the indentation of continued lines after @return or @param tags
>> isn't correct; the continued line should be indented.
>>
>> And there are a some very long lines that should be wrapped;
>> See +5637...   "If the SQL AS clause was not..."
>>
>> +5784 the indentation needs to be fixed.
>> +5840 also
>> +5988
>> +6265 and after...
>>   in many cases the lines that follow of "/*" do not align the "* ".
>>
>> CachedRowSetWriter.java:
>> - many method comment blocks are not aligned properly with the method of field that follows.
>> For example,
>> /**
>> * The <code>Connection</code> object that this writer will use to make a
>> * connection to the data source to which it will write data.
>> *
>> */
>>     private transient Connection con;
>>
>> 3)
>>
>> CachedRowSetWriter.java:
>> +1056:  The tag "@ param" should be tighted up to "@param"
>>
>> Thanks, Roger
>>
>>
>>
>> On 6/5/2015 8:51 AM, alexander stepanov wrote:
>>> Hello,
>>>
>>> Could you please review the fix
>>> http://cr.openjdk.java.net/~avstepan/8081517/webrev.00/
>>> for
>>> https://bugs.openjdk.java.net/browse/JDK-8081517
>>>
>>> Just some cleanup for docs.
>>>
>>> Thanks,
>>> Alexander
>
>
> 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
>
>
>




More information about the core-libs-dev mailing list