RFR: 8344024: Unnecessary Hashtable usage in RSAPSSSignature.DIGEST_LENGTHS

Sean Mullan mullan at openjdk.org
Wed Nov 13 15:02:37 UTC 2024


On Fri, 1 Nov 2024 12:15:36 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> The field `sun.security.rsa.RSAPSSSignature#DIGEST_LENGTHS` is modified only in `<clinit>`. It means we can use Immutable Map instead of Hashtable.
> Hashtable is legacy synchronized class, which have `synchronized` on its `get` method. It's not needed in our case.

@valeriepeng You should probably review this.

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

PR Comment: https://git.openjdk.org/jdk/pull/21823#issuecomment-2473862540


More information about the security-dev mailing list