RFR: 6415694 Clarification in Javadoc for java.rmi.AlreadyBoundException

[Jayashree Sk1]

Hi Stuart, 

Yup Contributed-by as picked from my email : jayashreesk . 
Will keep in mind about the complicated change-sets. 


Thanks
Jay
 
-----Stuart Marks <stuart.marks at oracle.com> wrote: -----
To: Jayashree Sk1 <jayashreesk at in.ibm.com>
From: Stuart Marks <stuart.marks at oracle.com>
Date: 05/01/2020 09:32AM
Cc: core-libs-dev at openjdk.java.net
Subject: [EXTERNAL] Re: RFR: 6415694 Clarification in Javadoc for java.rmi.AlreadyBoundException

Hi Jay,

OK, welcome! I'll sponsor this changeset for you. It's pretty simple, so I'll 
just paste the exported changeset below. (For complicated changesets, you'll 
have to ask a sponsor to host a webrev on cr.openjdk.java.net for you.) Please 
check the Contributed-by line in the changeset to make sure the attribution is 
correct. I just copied the From: line from your email, but if you want it to be 
different, please let me know.

I'll update the bug report with a pointer to this email thread.

   https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D6415694&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=rA_13vli8clPM_toR46hq8FVwH3XGr8z7cUfcQoqL-k&m=9YMcjPnAAEf40XGqmcEU_Uiyg_xCVlOU26-U56hEb4A&s=ueOfMpgrTsTZc7xg22AdzVskohCya_stq_WLTREzw24&e= 

Could I get a Reviewer to review this? Thanks.

(I noticed that the anachronistic <code> markup is used here instead of the 
preferred {@code ...} javadoc tag. However, <code> is used all over this 
package, so I didn't think that changing it in just one place would be helpful.)

I'll also note again that since this is merely an editorial wording change to 
the specification, I don't think it needs a CSR request.

Thanks,

s'marks





# HG changeset patch
# User smarks
# Date 1588290431 25200
#      Thu Apr 30 16:47:11 2020 -0700
# Node ID 1f5b3876d1500a2e77d6e39d079649964ba6dd26
# Parent  fe6548dc92de4c875467c968ee37e7c9bbee1697
6415694: Clarification in Javadoc for java.rmi.AlreadyBoundException
Reviewed-by: XXX
Contributed-by: Jayashree Sk1 <jayashreesk at in.ibm.com>

diff -r fe6548dc92de -r 1f5b3876d150 
src/java.rmi/share/classes/java/rmi/AlreadyBoundException.java
--- a/src/java.rmi/share/classes/java/rmi/AlreadyBoundException.java	Thu Apr 30 
15:21:15 2020 -0700
+++ b/src/java.rmi/share/classes/java/rmi/AlreadyBoundException.java	Thu Apr 30 
16:47:11 2020 -0700
@@ -26,8 +26,8 @@

  /**
   * An <code>AlreadyBoundException</code> is thrown if an attempt
- * is made to bind an object in the registry to a name that already
- * has an associated binding.
+ * is made to bind an object to a name that already
+ * has an associated binding in the registry.
   *
   * @since   1.1
   * @author  Ann Wollrath




On 4/29/20 10:24 PM, Jayashree Sk1 wrote:
> Thanks for the review comment Stuart. Yes, I will need a sponsor, this is my first time here in OpenJDK.
> 
> 
> Regards!
> Jay
>   
> -----Stuart Marks <stuart.marks at oracle.com> wrote: -----
> To: Jayashree Sk1 <jayashreesk at in.ibm.com>
> From: Stuart Marks <stuart.marks at oracle.com>
> Date: 04/30/2020 09:22AM
> Cc: core-libs-dev at openjdk.java.net
> Subject: [EXTERNAL] Re: RFR: 6415694 Clarification in Javadoc for java.rmi.AlreadyBoundException
> 
> The change looks fine. Although this is in a normative portion of the
> specification, the nature of the change is purely editorial, so I don't think it
> needs a CSR.
> 
> Do you need a sponsor?
> 
> s'marks
> 
> On 4/29/20 2:57 AM, Jayashree Sk1 wrote:
>> Hi All,
>>           Please find the below changes for the issues https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D6415694&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=rA_13vli8clPM_toR46hq8FVwH3XGr8z7cUfcQoqL-k&m=OXEC-w1xTWoJUyw_sRivRwIGmkHIUc3DllMsA_N3qIk&s=iesFFcE4NSOreLmf8vwdKzcdnFgjGE_SEERICPUFef4&e= .
>> It is a description change, which was already approved by the reviewer.
>>
>> Thanks!
>>
>> diff -r 59b5bd9a7168 src/java.rmi/share/classes/java/rmi/AlreadyBoundException.java
>> --- a/src/java.rmi/share/classes/java/rmi/AlreadyBoundException.java    Mon Mar 16 02:16:49 2020 -0400
>> +++ b/src/java.rmi/share/classes/java/rmi/AlreadyBoundException.java    Mon Mar 30 15:45:43 2020 +0530
>> @@ -26,8 +26,8 @@
>>    
>>    /**
>>     * An <code>AlreadyBoundException</code> is thrown if an attempt
>> - * is made to bind an object in the registry to a name that already
>> - * has an associated binding.
>> + * is made to bind an object to a name that already
>> + * has an associated binding in the registry.
>>     *
>>     * @since   1.1
>>     * @author  Ann Wollrath
>>
> 
>