On Tue, Feb 17, 2009 at 02:10:33AM +0100, Mikael Magnusson wrote: > 2009/2/17 Jeff King <peff@xxxxxxxx>: > > On Mon, Feb 16, 2009 at 10:52:05PM +0100, Johannes Schindelin wrote: > > > >> @@ -75,7 +77,12 @@ static int receive_pack_config(const char *var, const char *value, void *cb) > >> } > >> > >> if (!strcmp(var, "receive.denycurrentbranch")) { > >> - deny_current_branch = parse_deny_action(var, value); > >> + if (value && !strcasecmp(value, "updateinstead")) > >> + deny_current_branch = DENY_UPDATE_INSTEAD; > >> + else if (value && !strcasecmp(value, "detachinstead")) > > > > Config variables are already lower-cased. ;P > > But surely the values are not? Heh. Ouch. You are right, of course, and now I have been caught being wrong while making smart-ass comments, so I'll shut up. ;) -Peff -- 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