On Wed, 10 Feb 2021 06:01:59 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org>
wrote:
>> The API doc for Graphics2D.clip(shape s) claims that passing a null argument
>> would actually clear the existing clipping area, which is incorrect.
>> This statement is applicable only to G2D.setClip() and not for the clip()
>> method. G2D.clip() would throw a NullPointerException when it encounters a
>> null argument.
>> Updated spec to rectify this.
>
> Prasanta Sadhukhan has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Fix
test/jdk/java/awt/Graphics2D/TestNullClip.java line 52:
> 50: throw new RuntimeException("NPE is expected");
> 51: } catch (NullPointerException e) {
> 52: //expected
Why is this wrapping everything ? There's only one line here where you expect
an NPE and so you should only catch an exception from that one line.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2476
