Re: [PATCH] [merge-tracking] Refactoring the merge info indexing code.

Madan U Sreenivasan wrote:
> On Thu, 27 Jul 2006 22:33:50 +0530, Kamesh Jayachandran
> <kamesh@collab.net> wrote:
>
>> Thanks Madan.
>> Removed the redundant comment in preference to self documenting function
>> name.
>>
>> [[[
>> Patch by: Kamesh Jayachandran <kamesh@collab.net>
>>
>> Refactor 'mergeinfo indexing code'.
>>
>> * subversion/libsvn_fs_fs/fs_fs.c
>> (do_merge_info_indexing): New function.
>
> I mentioned this already... do_merge_info_indexing() is a longish and
> colloquial form on index_merge_info(). Is there any special reason you
> chose to retain this name?
>
> Regards,
> Madan.

I personally prefer 'do_merge_info_indexing()', though my preference is
not strong enough to stick with this name.

With regards
Kamesh Jayachandran

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jul 28 08:11:13 2006