commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From S├ębastien Brisard <sebastien.bris...@m4x.org>
Subject Re: [all] FindBugs' Switch statement found in class.method where default case is missing
Date Tue, 04 Sep 2012 14:44:12 GMT
Hello,

2012/9/4 James Carman <james@carmanconsulting.com>:
> Something like:
>
> throw new IllegalArgumentException("This should never happen because
> we are so smart we thought of every possibility in our case
> statement.");
>
> would suffice :)
>
Not that it really matters, since this is never going to occur, but I
think Bloch throws AssertionError in such cases.
I think that IllegalStateException would be more appropriate in this
case. But it really is for the sake of the argument anyway...

S├ębastien


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


Mime
View raw message