Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v6]

2023-03-06 Thread Ravali Yatham
On Mon, 6 Mar 2023 17:07:14 GMT, Mandy Chung wrote: >> Ravali Yatham has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added JavaLangAccess::getLoaderNameID(ClassLoader loader) > > Looks good. Thanks for the change. @mlchung @AlanBateman

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v6]

2023-03-06 Thread Mandy Chung
On Mon, 6 Mar 2023 11:08:42 GMT, Ravali Yatham wrote: >> Added specific class loader object to proxy IllegalArgumentException from >> which the class was not visible >> >> The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 > > Ravali Yatham has updated the pull request inc

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v6]

2023-03-06 Thread Ravali Yatham
> Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 Ravali Yatham has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v4]

2023-03-06 Thread Ravali Yatham
On Thu, 2 Mar 2023 20:00:24 GMT, Mandy Chung wrote: >>> Thanks @AlanBateman. Regarding the separator, Have seen this being used in >>> ClassLoader.java for nameAndId method. Hence used the same for consistency. >>> I've reformatted L886 now, Please kindly check. >> >> Thanks for checking, the

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v4]

2023-03-02 Thread Mandy Chung
On Thu, 2 Mar 2023 14:18:13 GMT, Alan Bateman wrote: >> src/java.base/share/classes/java/lang/reflect/Proxy.java line 889: >> >>> 887: : >>> Objects.toIdentityString(ld); >>> 888: throw new IllegalArgumentException(c.getName()

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v5]

2023-03-02 Thread Alan Bateman
On Thu, 2 Mar 2023 10:32:48 GMT, Ravali Yatham wrote: >> Added specific class loader object to proxy IllegalArgumentException from >> which the class was not visible >> >> The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 > > Ravali Yatham has updated the pull request inc

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v4]

2023-03-02 Thread Alan Bateman
On Wed, 1 Mar 2023 15:08:02 GMT, Alan Bateman wrote: >> Ravali Yatham has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add comments > > src/java.base/share/classes/java/lang/reflect/Proxy.java line 889: > >> 887:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v4]

2023-03-02 Thread Ravali Yatham
On Wed, 1 Mar 2023 15:08:02 GMT, Alan Bateman wrote: >> Ravali Yatham has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Add comments > > src/java.base/share/classes/java/lang/reflect/Proxy.java line 889: > >> 887:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v5]

2023-03-02 Thread Ravali Yatham
> Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 Ravali Yatham has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v4]

2023-03-01 Thread Alan Bateman
On Wed, 1 Mar 2023 10:10:12 GMT, Ravali Yatham wrote: >> Added specific class loader object to proxy IllegalArgumentException from >> which the class was not visible >> >> The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 > > Ravali Yatham has updated the pull request inc

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v4]

2023-03-01 Thread Ravali Yatham
> Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 Ravali Yatham has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v3]

2023-02-28 Thread Ravali Yatham
On Wed, 22 Feb 2023 07:24:48 GMT, Alan Bateman wrote: >> Ravali Yatham has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fix Indentation > > src/java.base/share/classes/java/lang/reflect/Proxy.java line 881: > >> 879: if (type

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v3]

2023-02-28 Thread Ravali Yatham
> Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 Ravali Yatham has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message [v2]

2023-02-28 Thread Ravali Yatham
> Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 Ravali Yatham has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message

2023-02-23 Thread Mandy Chung
On Wed, 22 Feb 2023 07:24:48 GMT, Alan Bateman wrote: >> Added specific class loader object to proxy IllegalArgumentException from >> which the class was not visible >> >> The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 > > src/java.base/share/classes/java/lang/reflect/

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message

2023-02-21 Thread Alan Bateman
On Sun, 19 Feb 2023 08:28:37 GMT, Ravali Yatham wrote: > Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 src/java.base/share/classes/java/lang/reflect/Proxy.

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message

2023-02-21 Thread Ravali Yatham
On Sun, 19 Feb 2023 16:12:39 GMT, liach wrote: >> Added specific class loader object to proxy IllegalArgumentException from >> which the class was not visible >> >> The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 > > src/java.base/share/classes/java/lang/reflect/Proxy.j

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message

2023-02-21 Thread liach
On Sun, 19 Feb 2023 08:28:37 GMT, Ravali Yatham wrote: > Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 src/java.base/share/classes/java/lang/reflect/Proxy.

Re: RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message

2023-02-21 Thread Ravali Yatham
On Sun, 19 Feb 2023 08:28:37 GMT, Ravali Yatham wrote: > Added specific class loader object to proxy IllegalArgumentException from > which the class was not visible > > The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 This contribution is on behalf of my employer, IBM,

RFR: 8302791: Add specific ClassLoader object to Proxy IllegalArgumentException message

2023-02-21 Thread Ravali Yatham
Added specific class loader object to proxy IllegalArgumentException from which the class was not visible The bug report for the same: https://bugs.openjdk.org/browse/JDK-8302791 - Commit messages: - Added specific ClassLoader object to Proxy IllegalArgumentException Changes: http