RFR [9] 8079075: some docs cleanup for CORBA - part 1
Alexander Stepanov
alexander.v.stepanov at oracle.com
Thu Apr 30 19:41:01 UTC 2015
Thanks!
----- Исходное сообщение -----
От: Roger.Riggs at Oracle.com
Кому: alexander.v.stepanov at oracle.com
Копия: core-libs-dev at openjdk.java.net
Отправленные: Четверг, 30 Апрель 2015 г 21:41:05 GMT +03:00 Москва, Санкт-Петербург, Волгоград
Тема: Re: RFR [9] 8079075: some docs cleanup for CORBA - part 1
Hi Alexander,
The improvements look good to me.
Thank you, Roger
On 4/30/2015 12:35 PM, alexander stepanov wrote:
> Hello Roger,
>
> Thank you for the notes; please see the updated webrev:
> http://cr.openjdk.java.net/~avstepan/8079075/webrev.01/
>
> Regards,
> Alexander
>
> On 30.04.2015 16:47, Roger Riggs wrote:
>> Hi Alexander,
>>
>> Thanks for working through all these changes.
>>
>> 0) Is there an upstream version of any of this code? Making even
>> cosmetic changes
>> would make future versions harder to merge.
>>
>> 1) For files that we don't typically generate javadoc for, it is
>> simpler to avoid adding unnecessary markup.
>> For example:
>>
>> old/src/java.corba/share/classes/com/sun/corba/se/impl/corba/AnyImpl.java:222+
>>
>> + * @return the {@code TypeCode} for the element in the Any
>>
>> There is not much value in the addition of {@code }.
>>
>> For these kind of files, the javadoc should be well formed and any
>> warnings/errors should be fixed.
>>
>> 2) a number of changes do not maintain the indentation, for example,
>> - * @result the InputStream to marshal value of Any out of.
>> + * @return the {@code InputStream} to marshal value of Any out of.
>>
>> Unless all of the indentation is going to be fixed in a file; it is
>> better to remain consistent with the per file style.
>>
>> 3) Removing degenerate @author tags is great. But adding an extra
>> blank line is not desirable.
>> For example,
>> old/src/java.corba/share/classes/com/sun/corba/se/impl/ior/GenericTaggedProfile.java:39
>>
>> -/**
>> - * @author
>> - */
>> +
>>
>> 4) For snippets of code, use {@code} instead of a plain or {@literal
>> ...}
>> These files don't that pattern extensively and its not important to
>> fix it everywhere since it is in the impl code.
>> But for example, it would make sense here:
>> old/src/java.corba/share/classes/com/sun/corba/se/impl/ior/ObjectAdapterIdNumber.java:31+
>>
>>
>> - * internally as arrays of the form { "OldRootPOA", "<number>" },
>> + * internally as arrays of the form { "OldRootPOA", "{@literal
>> <number>}" },
>>
>> Might be better as:
>> + * internally as arrays of the form {@code { "OldRootPOA",
>> "<number>" }},
>>
>> 5) The continuation of @param should be indented.
>>
>> src/java.corba/share/classes/com/sun/corba/se/impl/naming/pcosnaming/NamingContextImpl.java
>>
>> @@ -1204,14 +1202,13 @@
>> * This operation creates a URL based "iiopname://" format name
>> * from the Stringified Name of the object.
>> * @param addr internet based address of the host machine where
>> - * Name Service is running <p>
>> - * @param sn Stringified Name of the object <p>
>> *+ * Name Service is running*
>> + * @param sn Stringified Name of the object
>>
>>
>> 6) correct the punctuation since the line is being modified.
>> src/java.corba/share/classes/com/sun/corba/se/pept/transport/Acceptor.java
>>
>>
>> - * @return <code>true</code. if the <code>Acceptor</code> has been
>> + * @return <code>true</code>. if the <code>Acceptor</code> has been
>>
>> Would read better as:
>> + * @return <code>true</code> if the <code>Acceptor</code> has been
>>
>>
>> 7) I don't recognize the strings of the format dNNNNN but they seem
>> to be referring
>> to past bug reports and the authors who fixed them. I doubt they
>> have any value in
>> the code anymore. It would probably be fine to just drop the
>> extra '<>' symbols.
>> And around the author initials. <klr> --> klr
>>
>> Regards, Roger
>>
>>
>>
>> On 4/30/2015 7:56 AM, alexander stepanov wrote:
>>> P.S. Sorry, not sure if changes like "<d62023>" -> "{@literal
>>> <d62023>}" (see, e.g., ValueBoxGen24.java) are what expected; please
>>> correct me if I'm wrong.
>>>
>>> But something should be done; otherwise "<d62023>" is interpreted as
>>> an unknown HTML tag causing the errors.
>>>
>>> Thanks,
>>> Alexander
>>>
>>> On 30.04.2015 14:31, alexander stepanov wrote:
>>>> Hello,
>>>>
>>>> Could you please review the following fix
>>>> http://cr.openjdk.java.net/~avstepan/8079075/webrev.00/
>>>> for
>>>> https://bugs.openjdk.java.net/browse/JDK-8079075
>>>>
>>>> Just some HTML markup fix for CORBA.
>>>>
>>>> Thanks,
>>>> Alexander
>>>
>>
>
More information about the core-libs-dev
mailing list