On Sat, Dec 21, 2013 at 4:31 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Jeff King <peff@xxxxxxxx> writes: > >> /* here we care if we saw the prefix, as above */ >> if (parse_prefix(foo, prefix, &the_rest)) >> ... >> >> /* >> * and here we do not care, and just want to optionally strip the >> * prefix, and take the full value otherwise; we just have to ignore >> * the return value in this case. >> */ >> parse_prefix(foo, prefix, &foo); > > Sounds fine. I recall earlier somebody wanting to have a good name > for this thing, and I think foo_gently is *not* it (the name is > about adding a variant that does not die outright to foo that checks > and dies if condition is not right). > > starts_with(foo, prefix); > strip_prefix(foo, prefix, &foo); > > perhaps? I still need consensus on the name here guys, parse_prefix. remove_prefix or strip_prefix? If no other opinions i'll go with strip_prefix (Jeff's comment before parse_prefix() also uses "strip") -- Duy -- 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