1
1

One of the latest gcc version bark about a variable being use uninitialized. It was kind of right, because the

variable was protected by another one ... But with few modifications I get rid of this warning.

This commit was SVN r7189.
Этот коммит содержится в:
George Bosilca 2005-09-06 03:13:03 +00:00
родитель 5df64827c8
Коммит 648ef2ae5c
2 изменённых файлов: 13 добавлений и 16 удалений

Просмотреть файл

@ -46,7 +46,6 @@ int orte_gpr_proxy_deliver_notify_msg(orte_gpr_notify_message_t *msg)
orte_gpr_proxy_subscriber_t **subs, *sub;
orte_gpr_proxy_trigger_t *trig, **trigs;
size_t i, j, k, n;
bool processed;
int rc;
/* we first have to check if the message is a trigger message - if so,
@ -110,9 +109,8 @@ int orte_gpr_proxy_deliver_notify_msg(orte_gpr_notify_message_t *msg)
*/
subs = (orte_gpr_proxy_subscriber_t**)
(orte_gpr_proxy_globals.subscriptions)->addr;
processed = false;
for (j=0, k=0; !processed &&
k < orte_gpr_proxy_globals.num_subs &&
sub = NULL;
for (j=0, k=0; k < orte_gpr_proxy_globals.num_subs &&
j < (orte_gpr_proxy_globals.subscriptions)->size; j++) {
if (NULL != subs[j]) {
k++;
@ -121,17 +119,17 @@ int orte_gpr_proxy_deliver_notify_msg(orte_gpr_notify_message_t *msg)
if (NULL != subs[j]->name &&
0 == strcmp(data[i]->target, subs[j]->name)) {
sub = subs[j];
processed = true;
break;
}
} else if (data[i]->id == subs[j]->id) {
/* otherwise, see if id's match */
sub = subs[j];
processed = true;
break;
}
}
}
/* get here and not processed => not found, abort */
if (!processed) {
/* get here and not found => abort */
if (NULL == sub) {
ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
return ORTE_ERR_NOT_FOUND;
}

Просмотреть файл

@ -93,9 +93,8 @@ int orte_gpr_replica_deliver_notify_msg(orte_gpr_notify_message_t *msg)
*/
local_subs = (orte_gpr_replica_local_subscriber_t**)
(orte_gpr_replica_globals.local_subscriptions)->addr;
processed = false;
for (j=0, k=0; !processed &&
k < orte_gpr_replica_globals.num_local_subs &&
sub = NULL;
for (j=0, k=0; k < orte_gpr_replica_globals.num_local_subs &&
j < (orte_gpr_replica_globals.local_subscriptions)->size; j++) {
if (NULL != local_subs[j]) {
k++;
@ -104,18 +103,18 @@ int orte_gpr_replica_deliver_notify_msg(orte_gpr_notify_message_t *msg)
if (NULL != local_subs[j]->name &&
0 == strcmp(data[i]->target, local_subs[j]->name)) {
sub = local_subs[j];
processed = true;
break;
}
} else if (data[i]->id == local_subs[j]->id) {
/* otherwise, see if id's match */
sub = local_subs[j];
processed = true;
break;
}
}
}
/* get here and not processed => not found, abort */
if (!processed) {
/* get here and not found => abort */
if (NULL == sub ) {
ORTE_ERROR_LOG(ORTE_ERR_NOT_FOUND);
return ORTE_ERR_NOT_FOUND;
}