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

Reply via email to