Re: [PATCH 2/8] refs.c: rename the transaction functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 11, 2014 at 1:42 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Stefan Beller <sbeller@xxxxxxxxxx> writes:
>
>> Rename the transaction functions. Remove the leading ref_ from the
>> names and append _ref to the names for functions that create/delete/
>> update sha1 refs.
>>
>> The goal in the long term is to have different things running through
>> the transaction api, such as the .git/config file.
>>
>> The dream is to...
>
> I was trying to polish somebody else's topic that does not have
> anythning to do with refs/reflogs/transactions today, and had to
> suffer from conflicts from the textual difference between
> transaction_begin and ref_transaction_begin, which I feel is totally
> unnecessary at this moment.
>
> I'd really prefer to see this done long after all the dust from
> "ref" changes from various people have settled, especially given
> that we are not doing anything for that "dream" right now.
>
> Thanks.

Good point. For the last days I was trying to write less invasive
patches as I guessed this may become a problem.

Feel free to drop these patches,
I'll resend them later if they still make sense.
--
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




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]