From: Sverre Rabbelier Date: Wed, 05 Aug 2009 20:27:20 GMT Subject: Re: [PATCH 3/3] transport: don't show push status if --quiet is given Message-ID: In-Reply-To: <20090805202326.GC23226@coredump.intra.peff.net> Heya, On Wed, Aug 5, 2009 at 13:23, Jeff King 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? -- Cheers, Sverre Rabbelier