You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Julian Foad <ju...@wandisco.com> on 2010/03/11 18:58:28 UTC
Re: svn commit: r921503 - in
/subversion/trunk/subversion/libsvn_wc: adm_ops.c deprecated.c wc.h
On Wed, 2010-03-10, cmpilato@apache.org wrote:
> Author: cmpilato
> Date: Wed Mar 10 18:28:10 2010
> New Revision: 921503
>
> URL: http://svn.apache.org/viewvc?rev=921503&view=rev
> Log:
> Continue doing what it takes to get deprecated stuff into deprecated.c.
>
> * subversion/libsvn_wc/wc.h
> (svn_wc__process_committed_internal): New prototype (and real
> docstring!)
Hoorrah! Thank you, Mike.
> for what was adm_ops.c:process_committed_internal().
> (svn_wc__prop_array_to_hash): New prototype (for the function that
> was adm_ops.c:convert_to_hash()).
Excellent, too. Just one nit: it looks like you intended to rename the
argument "wcprop_changes" to "props".
- Julian
[...]
> -/* */
> -static apr_hash_t *
> -convert_to_hash(const apr_array_header_t *wcprop_changes,
> - apr_pool_t *result_pool)
> +apr_hash_t *
> +svn_wc__prop_array_to_hash(const apr_array_header_t *wcprop_changes,
> + apr_pool_t *result_pool)
[...]
> +/* Return a hash keyed by 'const char *' property names and with
> + 'svn_string_t *' values built from PROPS (which is an array of
> + pointers to svn_prop_t's) or to NULL if PROPS is NULL or empty.
> + PROPS items which lack a value will be ignored. If PROPS contains
> + multiple properties with the same name, each successive such item
> + reached in a walk from the beginning to the end of the array will
> + overwrite the previous in the returned hash.
> +
> + NOTE: While the returned hash will be allocated in RESULT_POOL, the
> + items it holds will share storage with those in PROPS.
> +
> + ### This is rather the reverse of svn_prop_hash_to_array(), except
> + ### that function's arrays contains svn_prop_t's, whereas this
> + ### one's contains *pointers* to svn_prop_t's. So much for
> + ### consistency. */
> +apr_hash_t *
> +svn_wc__prop_array_to_hash(const apr_array_header_t *wcprop_changes,
> + apr_pool_t *result_pool);
> +
> +
> /* Baton for svn_wc__compat_call_notify_func below. */
> typedef struct svn_wc__compat_notify_baton_t {
> /* Wrapped func/baton. */
>
>
Re: svn commit: r921503 - in
/subversion/trunk/subversion/libsvn_wc: adm_ops.c deprecated.c wc.h
Posted by Julian Foad <ju...@wandisco.com>.
On Wed, 2010-03-17, C. Michael Pilato wrote:
> Julian Foad wrote:
> > On Thu, 2010-03-11, I (Julian Foad) wrote:
> >> Excellent, too. Just one nit: it looks like you intended to rename the
> >> argument "wcprop_changes" to "props".
> >
> > Renamed in r924229.
>
> Thanks. For the record, I hadn't ignored your review -- I just had left the
> changes sitting uncommitted in my working copy yesterday when Distraction
> #1587 came to visit or something.
No problem. I just came back to it because I'm trying to understand the
code that uses it.
- Julian
Re: svn commit: r921503 - in
/subversion/trunk/subversion/libsvn_wc: adm_ops.c deprecated.c wc.h
Posted by Julian Foad <ju...@wandisco.com>.
On Wed, 2010-03-17, C. Michael Pilato wrote:
> Julian Foad wrote:
> > On Thu, 2010-03-11, I (Julian Foad) wrote:
> >> Excellent, too. Just one nit: it looks like you intended to rename the
> >> argument "wcprop_changes" to "props".
> >
> > Renamed in r924229.
>
> Thanks. For the record, I hadn't ignored your review -- I just had left the
> changes sitting uncommitted in my working copy yesterday when Distraction
> #1587 came to visit or something.
No problem. I just came back to it because I'm trying to understand the
code that uses it.
- Julian
Re: svn commit: r921503 - in /subversion/trunk/subversion/libsvn_wc:
adm_ops.c deprecated.c wc.h
Posted by "C. Michael Pilato" <cm...@collab.net>.
Julian Foad wrote:
> On Thu, 2010-03-11, I (Julian Foad) wrote:
>> Excellent, too. Just one nit: it looks like you intended to rename the
>> argument "wcprop_changes" to "props".
>
> Renamed in r924229.
Thanks. For the record, I hadn't ignored your review -- I just had left the
changes sitting uncommitted in my working copy yesterday when Distraction
#1587 came to visit or something.
--
C. Michael Pilato <cm...@collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Re: svn commit: r921503 - in /subversion/trunk/subversion/libsvn_wc:
adm_ops.c deprecated.c wc.h
Posted by "C. Michael Pilato" <cm...@collab.net>.
Julian Foad wrote:
> On Thu, 2010-03-11, I (Julian Foad) wrote:
>> Excellent, too. Just one nit: it looks like you intended to rename the
>> argument "wcprop_changes" to "props".
>
> Renamed in r924229.
Thanks. For the record, I hadn't ignored your review -- I just had left the
changes sitting uncommitted in my working copy yesterday when Distraction
#1587 came to visit or something.
--
C. Michael Pilato <cm...@collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Re: svn commit: r921503 - in
/subversion/trunk/subversion/libsvn_wc: adm_ops.c deprecated.c wc.h
Posted by Julian Foad <ju...@wandisco.com>.
On Thu, 2010-03-11, I (Julian Foad) wrote:
> Excellent, too. Just one nit: it looks like you intended to rename the
> argument "wcprop_changes" to "props".
Renamed in r924229.
- Julian
Re: svn commit: r921503 - in
/subversion/trunk/subversion/libsvn_wc: adm_ops.c deprecated.c wc.h
Posted by Julian Foad <ju...@wandisco.com>.
On Thu, 2010-03-11, I (Julian Foad) wrote:
> Excellent, too. Just one nit: it looks like you intended to rename the
> argument "wcprop_changes" to "props".
Renamed in r924229.
- Julian