Junio C Hamano <gitster@xxxxxxxxx> writes: > Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes: > >> Junio C Hamano <gitster@xxxxxxxxx> writes: >> >>> +/* >>> + * Call this function when you know "name" given by the end user must >>> + * name an object but it doesn't; the function _may_ die with a better >>> + * diagnostic message than "no such object 'name'", e.g. "Path 'doc' does not >>> + * exist in 'HEAD'" when given "HEAD:doc", or it may return in which case >>> + * you have a chance to diagnose the error further. >>> + */ >>> +void die_on_misspelt_object_name(const char *name, const char *prefix) >> >> It seems unusual to have a function named die_* that is not a noreturn >> function. I'd call it die_*_maybe, or diagnose_* instead. >> >> (but as the comment above documents the behavior, it's not terribly >> important, you may ignore my comment if you whish) > > I was hoping "on" may imply "if not misspelled, ignore and keep > going", but apparently that failed. I am not good at names. Ah, maybe I missed the "on". With your explanation, it makes sense. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- 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