]> rtime.felk.cvut.cz Git - orte.git/commitdiff
Remove useless condition
authorMichal Sojka <sojkam1@fel.cvut.cz>
Fri, 16 Jan 2015 12:10:13 +0000 (13:10 +0100)
committerMichal Sojka <sojkam1@fel.cvut.cz>
Fri, 16 Jan 2015 12:10:45 +0000 (13:10 +0100)
I did small investigation whether the second condition should compare
something else or not and it seems that no other comparison is necessary.
Therefore I believe that removing this rather than changing it to something
else is the correct thing to do.

Thanks to Martin Strunz for spotting this.

orte/liborte/objectUtils.c

index 43686aa83c0b9ecea7ea884f9217aa1e2823c521..09a5aae8a635596e7d97b2b4330e04739b08d015 100644 (file)
@@ -238,8 +238,8 @@ matchMulticastAddresses(ObjectEntryOID *o1,ObjectEntryOID *o2)
   AppParams    *ap1,*ap2;
   uint16_t      i,j;
   
-  if ((o1->guid.oid!=OID_APP) || 
-      (o1->guid.oid!=OID_APP)) return ORTE_FALSE;
+  if (o1->guid.oid!=OID_APP)
+    return ORTE_FALSE;
 
   ap1=o1->attributes;
   ap2=o2->attributes;