On Sat, Nov 6, 2010 at 9:04 AM, Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx> wrote: > diff --git a/tag.c b/tag.c > index 5f9626c..18a5142 100644 > --- a/tag.c > +++ b/tag.c > @@ -138,7 +138,8 @@ size_t parse_signature(const char *buf, unsigned long size) > Â{ > Â Â Â Âchar *eol; > Â Â Â Âsize_t len = 0; > - Â Â Â while (len < size && prefixcmp(buf + len, PGP_SIGNATURE)) { > + Â Â Â while (len < size && prefixcmp(buf + len, PGP_SIGNATURE) > + Â Â Â Â Â Â Â Â Â Â Â && prefixcmp(buf + len, PGP_MESSAGE)) { nit: I think this && should be in the end of the previous line. (Not sure what is the preferred style though, comments?). -- 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