Daniel Rall wrote:
> On Tue, 10 Oct 2006, Kamesh Jayachandran wrote:
>
>
>> I would see it would be more useful to have the following,
>> svn_range_create(start, end) which would have more use than this combo
>> (svn_mergeinfo__make).
>>
>
> An svn_merge_range_t only contains two field, start and end (see
> svn_types.h). Is there really enough value in adding a constructor
> function for such a simple data type? (We already have an
> svn_range_dup() API).
>
Sorry I was under the opinion each place we create 'svn_mergerange_t *'
we have some sequence of alloc mem, set start, set end. This I thought
could be replaced with one utility function. Unfortunately I could not
find one :).
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 Wed Oct 11 15:35:31 2006