On 06/05/2012 07:53 PM, Christoph Hellwig wrote: > - if (log->nr_nodes == -1) > + if (log->nr_nodes == -1) { > log->nr_nodes = epoch_log_read_remote(epoch, > (char *)log->nodes, > sizeof(log->nodes)); > + } > + if (log->nr_nodes == -1) { > + eprintf("Failed to read log for epoch %d\n", i); > + break; > + } > + epoch_log_read_remote() doesn't return -1, so no need to check it. Also need remove {} for one liner if clause. Thanks, Yuan |