From decff16805b2f1264674013234604af40245a585 Mon Sep 17 00:00:00 2001 From: Michal Sojka Date: Fri, 16 Jan 2015 13:10:13 +0100 Subject: [PATCH] Remove useless condition 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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/orte/liborte/objectUtils.c b/orte/liborte/objectUtils.c index 43686aa..09a5aae 100644 --- a/orte/liborte/objectUtils.c +++ b/orte/liborte/objectUtils.c @@ -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; -- 2.39.2