----- Original Message ----- From: "Jan Luehe" <[EMAIL PROTECTED]> To: "Tomcat Developers List" <[EMAIL PROTECTED]> Sent: Monday, November 01, 2004 3:41 PM Subject: Re: cvs commit: jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/connector RequestFacade.java LocalStrings.properties
> Remy Maucherat wrote: > > [EMAIL PROTECTED] wrote: > > > > > >>luehe 2004/11/01 14:38:44 > >> > >> Modified: catalina/src/share/org/apache/catalina/connector > >> RequestFacade.java LocalStrings.properties > >> Log: > >> Throw more meaningful exception (instead of NPE) if underlying request has been > >> recycled and attempt is made to access it via its facade > >> > > > > I think I always consistently refused this change (no use: if people who > > hack can't be bothered to look that up in the code, then I don't think > > they'll understand what your exception really means either), but I'll > > give up on that one. > > In this case, it's useful because rather than instinctively filing a bug > against Tomcat when seeing a NPE, people will be reminded to check their > code first, because they're obviously using Tomcat in a way it was not > intended to be used. > I agree with Remy: It's totally unnecessary, and gives somebody reading the code that the request can be null. The javadocs should probably be updated with something like: * @exception IllegalStateException If you are a total moron without a clue ;-) > We just ran into this internally. > > > Jan > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > This message is intended only for the use of the person(s) listed above as the intended recipient(s), and may contain information that is PRIVILEGED and CONFIDENTIAL. If you are not an intended recipient, you may not read, copy, or distribute this message or any attachment. If you received this communication in error, please notify us immediately by e-mail and then delete all copies of this message and any attachments. In addition you should be aware that ordinary (unencrypted) e-mail sent through the Internet is not secure. Do not send confidential or sensitive information, such as social security numbers, account numbers, personal identification numbers and passwords, to us via ordinary (unencrypted) e-mail.
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
