dev@javaserverfaces.java.net

Re: [REVIEW] Event Classes Should Throw IllegalArgumentException

From: Ed Burns <Ed.Burns_at_Sun.COM>
Date: Tue, 16 Dec 2008 11:03:09 -0800

>>>>> On Tue, 16 Dec 2008 01:24:47 -0500, Roger Kitain <Roger.Kitain_at_Sun.COM> said:

RK> M jsf-api/src/javax/faces/event/ViewMapCreatedEvent.java
RK> M jsf-api/src/javax/faces/event/ComponentSystemEvent.java
RK> M jsf-api/src/javax/faces/event/ViewMapDestroyedEvent.java
RK> M jsf-api/src/javax/faces/event/BeforeRenderEvent.java
RK> M jsf-api/src/javax/faces/event/AfterRestoreStateEvent.java
RK> M jsf-api/src/javax/faces/event/AfterAddToViewEvent.java
RK> M jsf-api/src/javax/faces/event/SystemEvent.java
RK> M jsf-api/src/javax/faces/event/InitialStateEvent.java

RK> JavaDocs: throw IllegalArgumentException instead of NullPointerException
RK> for consistency with other Faces event classes like ActionEvent.

r=edburns

-- 
| ed.burns_at_sun.com  | office: 408 884 9519 OR x31640
| homepage:         | http://ridingthecrest.com/