commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gary Gregory <>
Subject [all] FindBugs' Switch statement found in class.method where default case is missing
Date Tue, 04 Sep 2012 12:27:19 GMT
Hi All:

FindBugs can give warnings like:

Switch statement found in
org.apache.commons.codec.binary.Base32.decode(byte[], int, int,
BaseNCodec$Context) where default case is missing

In this case for [codec], it looks like the code was carefully constructed
and that no default clause is needed.

In these cases for any component, this FindBugs issue feels like a style
issue, is it worth changing the code to add a default clause like:

      // ok to fall through for other values.

Or does this feel like noise to you all?


E-Mail: |
JUnit in Action, 2nd Ed: <http://goog_1249600977>
Spring Batch in Action: <>

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message