<Swing Dev> [11][JDK-8153532] RFR: Add @throws NPE javadoc to UIManager.setLookAndFeel(String) method description

[Pankaj Bansal]

Hi Sergey,

<<This test will check that this method works according to its specification. Previously NPE was not specified and <<implementation of the method could or could not throw NPE on null, so I doubt that any existing tests checks null <<parameter.

Done 
Webrev : http://cr.openjdk.java.net/~pbansal/8153532/webrev.02/

Regards,
Pankaj Bansal

-----Original Message-----
From: Sergey Bylokhov 
Sent: Thursday, April 12, 2018 4:18 AM
To: Pankaj Bansal; swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> [11][JDK-8153532] RFR: Add @throws NPE javadoc to UIManager.setLookAndFeel(String) method description

On 11/04/2018 02:55, Pankaj Bansal wrote:
> I have checked all the functions and I could not find any other API with wrong description.

ok

> <<The test to prove that, is welcome as well.
> I think I won't be able to write a test which fails before the fix and passes afterwards as it is just a documentation change. Can you please tell me what are the requirements from the test and I will try to write one?

This test will check that this method works according to its specification. Previously NPE was not specified and implementation of the method could or could not throw NPE on null, so I doubt that any existing tests checks null parameter.


--
Best regards, Sergey.