On Tue, 31 May 2022 18:27:29 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:
> Looks good to me. I'm not sure the `requireNonNull` method carries its own > weight given that one `getSnippetById` method could by implemented by calling > the other, but that's a minor stylistic concern. OK, I looked at this again, and now see what you mean. I'll look at simplifying this. ------------- PR: https://git.openjdk.java.net/jdk/pull/8892