On Sat, 30 Apr 2022 16:43:32 GMT, Andrey Turbanov <aturba...@openjdk.org> wrote:
> Instead of pair `TreeMap.containsKey`/`TreeMap.put` method calls, we can use > single call `TreeMap.putIfAbsent` and check result for nullness. > Only non-null values are put into `Map<String, DomainEntry> domainEntries`. > https://urldefense.com/v3/__https://github.com/openjdk/jdk/blob/53a0acee06eb32fba700967c9a34d37ea42f7a99/src/java.base/share/classes/sun/security/provider/PolicyParser.java*L205-L214__;Iw!!ACWV5N9M2RV99hQ!MYjrZeeC3SvB-PXMjtJ1MKbKVlyU-a4FU-6T-YpF2xuNMD8V4t-D72Yerko2VutSI2IROskm_vBLAiURjVc02YrJaw$ > This pull request has now been integrated. Changeset: 7acdcc10 Author: Andrey Turbanov <aturba...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/7acdcc1034fb9529ee7b617889d38b0e557a20ec Stats: 4 lines in 1 file changed: 0 ins; 2 del; 2 mod 8288329: Avoid redundant TreeMap.containsKey call in PolicyParser.read Reviewed-by: mullan ------------- PR: https://git.openjdk.org/jdk/pull/8486
