On Wed, Aug 5, 2009 at 8:27 PM, Sverre Rabbelier<srabbelier@xxxxxxxxx> wrote: > On Wed, Aug 5, 2009 at 13:23, Jeff King<peff@xxxxxxxx> wrote: >> + for (; ref; ref = ref->next) { >> + switch (ref->status) { >> + case REF_STATUS_NONE: >> + case REF_STATUS_UPTODATE: >> + case REF_STATUS_OK: >> + break; >> + default: >> + return 1; >> + } >> + } > > Is my C getting sloppy or is this a non-looping loop? AFAICS it either > return 1's on the first run, or breaks? The "break" in this case breaks out of the switch, not the for. Avery -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html