Can we change all occurrences of @exception -> @throws in this method ?
In fact this method appears to be the only one in this class to use @exception
so it is doubly anomalous.

-phil.

On 04/10/2018 04:07 AM, Krishna Addepalli wrote:

Hi Pankaj,

The change looks fine to me.

Thanks,

Krishna

*From:* Pankaj Bansal
*Sent:* Tuesday, April 10, 2018 4:27 PM
*To:* swing-dev@openjdk.java.net
*Subject:* <Swing Dev> [11][JDK-8153532] RFR: Add @throws NPE javadoc to UIManager.setLookAndFeel(String) method description

Hi All,

Please review a very simple fix for documentation change enhancement:

JDK-8153532: https://bugs.openjdk.java.net/browse/JDK-8153532

Webrev: http://cr.openjdk.java.net/~pbansal/8153532/webrev.00/ <http://cr.openjdk.java.net/%7Epbansal/8153532/webrev.00/>

CSR: https://bugs.openjdk.java.net/browse/JDK-8201363

UIManager.setLookAndFeel (String) throws NPE if the className is null. But this is not updated in the API description. Made changes for the same.

Regards,

Pankaj Bansal


Reply via email to