On Wed, 12 Jan 2022 15:04:04 GMT, Kevin Walls <kev...@openjdk.org> wrote:

> The HotSpotAgent.java setupDebugger method has a commmented out section 
> relating to possibly using RMISecurityManager.
> The comment is from pre-jdk7.  As RMISecurityManager has been deprecated for 
> a while the comments should be removed.
> 
> This is a comment-only change, no impact on what we build, so seems trivial.
> 
> This is slightly more relevant today as with JEP411 and plans to remove the 
> Security Mananger, it is natural to search the source for references to 
> Security Manager.  Removing a false-positive from those results is a good 
> thing.

Hi Kevin,
This looks okay to me.
Thanks,
Serguei

-------------

Marked as reviewed by sspitsyn (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7050

Reply via email to