julianfoad_at_tigris.org writes:
> Log:
> Document whether constructors make deep or shallow copies of data passed to
> them by reference.
>
> * subversion/include/svn_fs.h
> (svn_fs_create_access): Document that it makes a deep copy of 'username',
>
> * subversion/include/svn_string.h
> Document that all string functions make deep copies of input strings.
>
> * subversion/include/svn_wc.h
> (svn_wc_create_notify, svn_wc_conflict_description_create_text,
> svn_wc_conflict_description_create_prop, svn_wc_create_conflict_result):
> Document that each of these makes a shallow copy of pointer arguments.
Sorry, I know I should have responded to your earlier request for
comment on this (but I already told you about my lost day looking for my
wireless modem, right?)
Anyway, the change looks good to me. It wouldn't have made sense to do
deep copies of the paths unless we were also doing deep copies of the
access batons. In the one case where it does make sense
(svn_fs_create_access), we make the deep copy.
+1
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-05 21:20:41 CEST