RFR [9] 8079342: some docs cleanup for CORBA - part 2
alexander stepanov
alexander.v.stepanov at oracle.com
Tue May 5 16:30:13 UTC 2015
> _variableName in the public javadocs
Please see
http://cr.openjdk.java.net/~avstepan/8079342/webrev.00/src/java.corba/share/classes/com/sun/corba/se/impl/io/ValueHandlerImpl.java.udiff.html
(please update the page as I didn't prepare a new webrev). Hopefully
that do not harm the code...
Thanks,
Alexander
On 05.05.2015 18:46, Lance Andersen wrote:
> Hi Alexander,
>
> Thank you, I guess I am not a fan of _variableName in the public
> javadocs as it is not something we normally see.
>
> Understand less is more, especially with CORBA.
>
> On May 5, 2015, at 11:41 AM, alexander stepanov
> <alexander.v.stepanov at oracle.com
> <mailto:alexander.v.stepanov at oracle.com>> wrote:
>
>> Hello Lance,
>>
>> It just seemed to be less hazardous. E.g., method 'readValue' (in
>> ValueHandlerImpl) contains inner variable 'in' and parameter '_in'
>> being documented (also, '_sender' - 'sender').
>>
>> But of course the names could be swapped, if desired. I lazily
>> preferred not to touch the code (just in case...).
>>
>> Regards,
>> Alexander
>>
>>
>> On 05.05.2015 18:25, Lance Andersen wrote:
>>> Hi Alexander,
>>>
>>> I just started to look at this and have a question:
>>>
>>>
>>> /**
>>> * Writes the value to the stream using java semantics.
>>> - * @param out The stream to write the value to
>>> + * @param _out The stream to write the value to
>>> * @param value The value to be written to the stream
>>> **/
>>> public void writeValue(org.omg.CORBA.portable.OutputStream _out,
>>>
>>>
>>> Is there a reason that you did not change the parameter name to
>>> "out" vs changing the @param tag to "_out" ?
>>>
>>> Best,
>>> lance
>>>
>>>
>>> On May 5, 2015, at 11:21 AM, alexander stepanov
>>> <alexander.v.stepanov at oracle.com
>>> <mailto:alexander.v.stepanov at oracle.com>
>>> <mailto:alexander.v.stepanov at oracle.com>> wrote:
>>>
>>>> Hello,
>>>>
>>>> Could you please review the following fix
>>>> http://cr.openjdk.java.net/~avstepan/8079342/webrev.00/
>>>> <http://cr.openjdk.java.net/%7Eavstepan/8079342/webrev.00/>
>>>> <http://cr.openjdk.java.net/%7Eavstepan/8079342/webrev.00/>
>>>> for
>>>> https://bugs.openjdk.java.net/browse/JDK-8079342
>>>>
>>>> Just some HTML markup fix for CORBA.
>>>>
>>>> Thanks,
>>>> Alexander
>>>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>>> <mailto:Lance.Andersen at oracle.com>
>>>
>>>
>>>
>>
>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif><http://oracle.com/us/design/oracle-email-sig-198324.gif>
> <http://oracle.com/us/design/oracle-email-sig-198324.gif>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 <mailto:Lance.Andersen at oracle.com>
>
>
>
More information about the core-libs-dev
mailing list