Christian Couder <chriscool@xxxxxxxxxxxxx> writes: > This patch parses the trailer command line arguments > and put the result into an arg_tok doubly linked > list. No the patch doesn't parse anything ;-). "Parse the command line arguments and ...". > +static void parse_trailer(struct strbuf *tok, struct strbuf *val, const char *trailer) > +{ > + const char *end = strchr(trailer, '='); > + if (!end) > + end = strchr(trailer, ':'); How would you explain the behaviour of the above code for this input? frotz: nitfol=xyzzy Perhaps strcspn()? -- 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