On Wed, 25 May 2022 19:38:13 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:

> Please review a simple change for `SnippetUtils` to throw a checked exception 
> if a snippet cannot be found.

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.

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

Marked as reviewed by hannesw (Reviewer).

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

Reply via email to