On Fri, 12 Feb 2021 14:42:30 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review comment fix. Test updated > > src/java.desktop/share/classes/java/awt/Graphics2D.java line 1204: > >> 1202: * argument, the specified {@code Shape} becomes the new >> 1203: * user clip. Since this method intersects the specified shape >> 1204: * with the current clip, it will throw NPE for a null shape > > Shall NPE be spelled out as `{@code NullPointerException}`? > > Also should it rather be, “…it <del>will</del> throws NPE…”? Done..As for the statement change, I think "will throw" is more grammatically correct" so kept it unchanged. ------------- PR: https://git.openjdk.java.net/jdk/pull/2476