RFR: JDK-8284112 Minor cleanup could be done in javax.crypto
Bradford Wetmore
wetmore at openjdk.java.net
Wed Apr 13 06:07:11 UTC 2022
On Tue, 12 Apr 2022 21:59:09 GMT, Mark Powers <duke at openjdk.java.net> wrote:
> JDK-8284112 Minor cleanup could be done in javax.crypto
src/java.base/share/classes/javax/crypto/ExemptionMechanism.java line 142:
> 140: * @see java.security.Provider
> 141: */
> 142: public static ExemptionMechanism getInstance(String algorithm)
Others might disagree with this comment, but I would encourage you not to make these changes throughout your PR. (@jddarcy ?)
Even though a static method is implicitly final and IJ is correct, the final keyword does prevent subclasses from inadvertently using the same method signature.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8214
More information about the security-dev
mailing list