Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#423 Add suppressLogLoadErrors property to AbstractTypeAwareCheck

release_4.2
closed
Oliver Burn
5
2012-10-10
2006-05-19
Clint Stotesbery
No

I've used the logLoadErrors property for the
JavadocMethod and RedundantThrows checks. It converted
the RunTimeException messages that I was getting into
Checkstyle warnings. Now I had 25 errors from the
RunTimeExceptions but that turned into 118 Checkstyle
warnings. I llike the logLoadErrors property because
now all the files are being checked regardless of
there being a problem of finding the exception class.
This uncovered additional violations for me.

The part that isn't as nice is now I have 118
Checkstlye warnings that aren't actually violations. I
need to use those two checks but it would be nice to
have another property added to
AbstractTypeAwareCheck.java that would supress those
message from showing up. This way I can set
logLoadErrors to true to allow Checkstyle to continue
checking the files where it encounters the
RunTimeException and then set the new property to true
to not have those messages show up as Checkstyle
violations/warnings.

Please add a property for this functionality. Maybe
suppressLogLoadErrors or suppressLoadErrors or
suppressRunTimeExceptionWarnings or
suppressTypeAwareWarnings, you get the idea. It should
probably be set to false by default so users will see
the warnings and then choose to ignore them by setting
the new property to true.

Here is all the relevant discussions relatd to this
issue:

http://sourceforge.net/forum/forum.php?
thread_id=1473762&forum_id=274377

http://sourceforge.net/tracker/index.php?
func=detail&aid=1462282&group_id=29721&atid=397078

http://sourceforge.net/tracker/?
func=detail&atid=559494&aid=1464012&group_id=80344

http://sourceforge.net/tracker/index.php?
func=detail&aid=1467156&group_id=29721&atid=397078

Discussion

  • Oliver Burn
    Oliver Burn
    2006-06-17

    Logged In: YES
    user_id=218824

    Patch applied - thanks.