On Wed, 16 Dec 2020 19:44:04 GMT, Mandy Chung wrote:
>> Mandy is correct, the bigger picture here is accessibility. Code in the
>> java.instrument module should be able to invoke the premain or agentmain
>> without needing to suppress access checks. It's a perquisite to supporting
>> the deplo
On Wed, 16 Dec 2020 08:18:49 GMT, Alan Bateman wrote:
> So PR1694 is really just about aligning the spec so that the premain method
> is public. However, it does not fix the overall accessibility issue.
OK I see that JDK-5070281 changes to allow the agent class to be non-public as
the main cla
On Wed, 16 Dec 2020 08:11:44 GMT, Serguei Spitsyn wrote:
>> Thanks, David!
>> Yes, I was also thinking that the setAccessible has to be remained after all
>> the checks.
>
> I've pushed an update with the following changes:
> - InstrumentationImpl.java update with the process sequence suggested
On Wed, 16 Dec 2020 02:23:33 GMT, Serguei Spitsyn wrote:
>> David, thank you for catching this. I'm probably missing something here.
>> If the agent class is not public then the `m.canAccess(null)` check is not
>> passed and IAE is thrown with the message:
>> `Exception in thread "main" java.l
Hi Mandy,
On 16/12/2020 12:54 pm, Mandy Chung wrote:
On 12/15/20 5:50 PM, David Holmes wrote:
On 16/12/2020 11:35 am, Serguei Spitsyn wrote:
On Wed, 16 Dec 2020 00:50:04 GMT, David Holmes
wrote:
The agent class doesn't have to be public it just has to be accessible.
The premain method sho
On 12/15/20 5:50 PM, David Holmes wrote:
On 16/12/2020 11:35 am, Serguei Spitsyn wrote:
On Wed, 16 Dec 2020 00:50:04 GMT, David Holmes
wrote:
The agent class doesn't have to be public it just has to be accessible.
The premain method should be queried for public modifier rather than
just re
On Wed, 16 Dec 2020 01:32:53 GMT, Serguei Spitsyn wrote:
>> The agent class doesn't have to be public it just has to be accessible.
>>
>> The premain method should be queried for public modifier rather than just
>> relying on a failed invocation request.
>
> David, thank you for catching this.
On 16/12/2020 11:35 am, Serguei Spitsyn wrote:
On Wed, 16 Dec 2020 00:50:04 GMT, David Holmes wrote:
The agent class doesn't have to be public it just has to be accessible.
The premain method should be queried for public modifier rather than just
relying on a failed invocation request.
Davi
On Wed, 16 Dec 2020 00:50:04 GMT, David Holmes wrote:
>> Mandy, thank you for the suggestion.
>> I'll retarget the bug and CSR to jdk 17 as nobody is objecting.
>>
>> Also, wanted to make it clear about Exception messages that are provided for
>> two different cases.
>>
>> The test java/lang/i
On Wed, 16 Dec 2020 00:15:36 GMT, Serguei Spitsyn wrote:
>> Alan, David and Many, thank you for the comments!
>> I'll prepare an update according to the recent requests from you.
>> One question is if I need to clone this PR to the JDK 16 fork:
>> https://github.com/openjdk/jdk16
>> It depends
On Mon, 14 Dec 2020 22:45:26 GMT, Serguei Spitsyn wrote:
>> Changes requested by dholmes (Reviewer).
>
> Alan, David and Many, thank you for the comments!
> I'll prepare an update according to the recent requests from you.
> One question is if I need to clone this PR to the JDK 16 fork:
> https
On 12/14/20 2:47 PM, Serguei Spitsyn wrote:
On Mon, 14 Dec 2020 02:39:30 GMT, David Holmes wrote:
Serguei Spitsyn has updated the pull request incrementally with one additional
commit since the last revision:
added 8165276 to @bug list of impacted tests
Changes requested by dholmes (Re
On Mon, 14 Dec 2020 02:39:30 GMT, David Holmes wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> added 8165276 to @bug list of impacted tests
>
> Changes requested by dholmes (Reviewer).
Alan, David and Many, tha
On Sat, 12 Dec 2020 09:48:27 GMT, Alan Bateman wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> added 8165276 to @bug list of impacted tests
>
> src/java.instrument/share/classes/sun/instrument/InstrumentationImp
On Sat, 12 Dec 2020 01:29:28 GMT, Serguei Spitsyn wrote:
>> This change have been already reviewed by Mandy, Sundar, Alan and David.
>> Please, see the jdk 15 review thread:
>>
>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2020-June/031998.html
>>
>> Now, the PR approval is nee
On Sat, 12 Dec 2020 01:29:28 GMT, Serguei Spitsyn wrote:
>> This change have been already reviewed by Mandy, Sundar, Alan and David.
>> Please, see the jdk 15 review thread:
>>
>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2020-June/031998.html
>>
>> Now, the PR approval is nee
> This change have been already reviewed by Mandy, Sundar, Alan and David.
> Please, see the jdk 15 review thread:
>
> http://mail.openjdk.java.net/pipermail/serviceability-dev/2020-June/031998.html
>
> Now, the PR approval is needed.
> The push was postponed because the CSR was not approved at
17 matches
Mail list logo