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

Reply via email to