-----Original Message----- From: Phil Steitz [mailto:[EMAIL PROTECTED] 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: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]