commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz" <p...@steitz.com>
Subject Re: [math] @throws IllegalArgumentException
Date Sat, 14 Jun 2003 22:13:01 GMT
Brent Worden wrote:
>>-----Original Message-----
>>From: Phil Steitz [mailto:phil@steitz.com]
>>Sent: Saturday, June 14, 2003 4:21 PM
>>To: Jakarta Commons Developers List
>>Subject: [math] @throws IllegalArgumentException
>>
>>
>>I am dutifully cleaning up the CheckStyle warnings in my code and I am
>>hesitating to remove @throws IllegalArgumentException, which CheckStyle
>>currently complains about. I am a little ambivalent about this.  There
>>is a property (checkstyle.javadoc.checkUnusedThrows) that we can set to
>>make it ignore these. I notice that [lang] has this set to false. I
>>actually prefer to leave these "unused throws tags" in. Any strong
>>opinions on this?
>>
>>Phil
>>
> 
> 
> I have no problem with it.
> 
> A better alternative might be to explicitly add runtime exceptions to the
> throws clause of methods.  This doesn't effect callers as they're still not
> required to add try-catch blocks. Plus, we could then leave the check enable
> to catch any actual violations.

Much better idea. Thanks. Thought I had tried that, but I now see that 
it works.

Phil

> 
> Brent Worden
> http://www.brent.worden.org
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 




---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message