On Thu, 2 Jun 2022 01:53:25 GMT, Hai-May Chao <hc...@openjdk.org> wrote:

> Please review a small fix in CryptoPolicyParser class that it should not pass 
> “processedPermissions” parameter by value.
> Ran MACH5 tier1 and tier2 without failures.

The fix looks fine. Could you add a test for this in the 
`test/jdk/javax/crypto/CryptoPermissions/` directory? Something that checks if 
a `ParsingException` is thrown if there are inconsistent entries?

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

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

Reply via email to