Jeff King <peff@xxxxxxxx> writes: >> > +A config callback should return 0 for success, or -1 if the variable >> > +could not be parsed properly. >> >> This matches what I have always thought, but I think I recently saw a >> series that adds callbacks that return 1 to mean "I have understood this >> variable, so callers should not look at it any more". It felt wrong, but >> I did not find anything in the config.c API framework to prvent such a >> local calling convention. > > ... > The "1 means I understood this" convention is used by userdiff_config. I > don't like that it is unlike every other config callback,... > Looking at the code again, though, ... > Hmm. Yeah. The userdiff calling convention dates back to late 2008.... > So I think we could go back and simplify the userdiff_config code now. I remembered where I saw the new "offender"; it was nd/columns topic (Cc'ing Nguyễn). -- 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