On Wed, 2005-09-14 at 17:56 -0700, Anthony Green wrote:
> This doesn't really fix a bug.  FindBugs was complaining about a
> potential null dereference.  This patch essentially turns a potential
> NullPointerException into a ClassCastException, which is no worse - and
> quiets FindBugs on this issue.

Both are wrong. If result is null (both conditions fail) the method
should throw a NoSuchAlgorithmException.

Cheers,

Mark

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to