Re: svn commit: r1309283 - in /subversion/trunk/subversion: include/private/svn_client_private.h libsvn_client/commit_util.c libsvn_client/copy.c libsvn_client/merge.c libsvn_client/mergeinfo.c libsvn_client/util.c
Greg Stein wrote:
>> Convert svn_client__wc_node_get_origin() to use svn_client__pathrev_t for
>> its output.
>> * subversion/include/private/svn_client_private.h,
>> (svn_client__wc_node_get_origin): Use pathrev_t for the output.
> This revision certainly shows how pathrev_t is a simplifying concept. Very nice!
> That said: I'd recommend being wary in your work about the struct being an *input* param in public APIs. There are seriously heavy internal constraints on the struct members. (eg. pass a struct with a NULL uuid, or even a non-matching one)
> I don't have a recommendation right now for what happens if the struct is made public. I just wanted to raise a yellow flag. It seems best to keep it very private because of the difficult constraints/preconditions in its members.
Right, good point. I'd really like to see this (or something similar) become more widespread so I hope we can find a satisfactory solution.
Received on 2012-04-04 15:18:09 CEST
This is an archived mail posted to the Subversion Dev