On Fri, 1 Nov 2024 12:15:36 GMT, Andrey Turbanov <aturba...@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. Look good to me. Thanks for the improvement~ ------------- Marked as reviewed by valeriep (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/21823#pullrequestreview-2434255433
