I'm checking this in.
Another bug found by FindBugs. The code in question is obviously
useless; the sense of the test was wrong.
Tom
Index: ChangeLog
from Tom Tromey <[EMAIL PROTECTED]>
PR classpath/22992:
* javax/security/sasl/Sasl.java (getSaslClientFactories): Fix
sense of test.
(getSaslServerFactories): Likewise.
Index: javax/security/sasl/Sasl.java
===================================================================
RCS file: /cvsroot/classpath/classpath/javax/security/sasl/Sasl.java,v
retrieving revision 1.5
diff -u -r1.5 Sasl.java
--- javax/security/sasl/Sasl.java 2 Jul 2005 20:32:46 -0000 1.5
+++ javax/security/sasl/Sasl.java 26 Jul 2005 22:30:41 -0000
@@ -448,7 +448,7 @@
HashSet names = new HashSet();
Provider[] providers = Security.getProviders();
Iterator it;
- if (providers == null)
+ if (providers != null)
{
Provider p;
String key;
@@ -640,7 +640,7 @@
HashSet names = new HashSet();
Provider[] providers = Security.getProviders();
Iterator it;
- if (providers == null)
+ if (providers != null)
{
Provider p;
String key;
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches
