From: Mikael Magnusson <mikachu@gmail.com>
Date: Tue, 17 Feb 2009 01:10:33 GMT
Subject: Re: [DRY HUMOR PATCH] Add a few more values for receive.denyCurrentBranch
Message-ID: <237967ef0902161710k4672e972rdffab0357b8872a8@mail.gmail.com>
In-Reply-To: <20090217005824.GA23892@coredump.intra.peff.net>

2009/2/17 Jeff King <peff@peff.net>:
> 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?

-- 
Mikael Magnusson