From: Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date: Sat, 10 Dec 2011 15:40:22 GMT
Subject: Re: [PATCH 1/3] Convert resolve_ref+xstrdup to new resolve_refdup function
Message-ID: <CACsJy8C0CJyHdtWJ5QVqX9ksHWgdBpm6XekQ+mZP4sxBVA_8vQ@mail.gmail.com>
In-Reply-To: <20111210131503.GI22035@elie.hsd1.il.comcast.net>

2011/12/10 Jonathan Nieder <jrnieder@gmail.com>:
>> --- a/builtin/revert.c
>> +++ b/builtin/revert.c
>> @@ -901,7 +901,7 @@ static int rollback_single_pick(void)
>>       if (!file_exists(git_path("CHERRY_PICK_HEAD")) &&
>>           !file_exists(git_path("REVERT_HEAD")))
>>               return error(_("no cherry-pick or revert in progress"));
>> -     if (!resolve_ref("HEAD", head_sha1, 0, NULL))
>> +     if (read_ref_full("HEAD", head_sha1, 0, NULL))
>>               return error(_("cannot resolve HEAD"));
>>       if (is_null_sha1(head_sha1))
>>               return error(_("cannot abort from a branch yet to be born"));
>
> Unrelated change that snuck in, I assume?

Yeah that slipped in. It should be part of c689332 (Convert many
resolve_ref() calls to read_ref*() and ref_exists() - 2011-11-13). I
guess either I missed it or it was a new call site after that patch.
Split it out as a separate patch?
-- 
Duy