RFR: 8344366: Remove Security Manager dependencies from javax.net.ssl and sun.security.ssl packages [v2]

Sean Mullan mullan at openjdk.org
Fri Nov 22 16:43:18 UTC 2024


On Thu, 21 Nov 2024 23:21:35 GMT, Alexey Bakhtin <abakhtin at openjdk.org> wrote:

>> Why do you think it is an API change? It produces the exact same javadoc. Technically it's not related, but minor cleanup changes like this are ok in my opinion.
>
> Yes, you are right, javadoc is the same. So, not entirely correct about API.
> But the same exact place in the KeyManagerFactory is not modified. I do not know the exact policy about references in the javadoc. I prefer to have a full class name if not in the same package. But, from my point of view, it is better to have consistent changes in similar places.

Yes, I think it makes sense to revert it for consistency. Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22301#discussion_r1854259548


More information about the net-dev mailing list