Re: svn commit: r1068169 - /subversion/trunk/tools/server-side/svn-populate-node-origins-index.c

[Daniel Shahaf]

hwri...@apache.org wrote on Mon, Feb 07, 2011 at 22:09:15 -0000:
> Author: hwright
> Date: Mon Feb  7 22:09:15 2011
> New Revision: 1068169
> 
> URL: http://svn.apache.org/viewvc?rev=1068169&view=rev
> Log:
> * tools/server-side/svn-populate-node-origins-index.c
>   (index_revision_adds): Update a deprecated function call, simplifying the
>     code a bit.
> 
> Modified:
>     subversion/trunk/tools/server-side/svn-populate-node-origins-index.c
> 
> Modified: subversion/trunk/tools/server-side/svn-populate-node-origins-index.c
> URL: 
> http://svn.apache.org/viewvc/subversion/trunk/tools/server-side/svn-populate-node-origins-index.c?rev=1068169&r1=1068168&r2=1068169&view=diff
> ==============================================================================
> --- subversion/trunk/tools/server-side/svn-populate-node-origins-index.c 
> (original)
> +++ subversion/trunk/tools/server-side/svn-populate-node-origins-index.c Mon 
> Feb  7 22:09:15 2011
> @@ -77,7 +77,7 @@ index_revision_adds(int *count, svn_fs_t
>  
>    *count = 0;
>    SVN_ERR(svn_fs_revision_root(&root, fs, revision, pool));
> -  SVN_ERR(svn_fs_paths_changed(&changes, root, pool));
> +  SVN_ERR(svn_fs_paths_changed2(&changes, root, pool));
>  
>    /* No paths changed in this revision?  Nothing to do.  */
>    if (apr_hash_count(changes) == 0)
> @@ -88,7 +88,7 @@ index_revision_adds(int *count, svn_fs_t
>      {
>        const void *path;
>        void *val;
> -      svn_fs_path_change_t *change;
> +      svn_fs_path_change2_t *change;
>  
>        svn_pool_clear(subpool);
>        apr_hash_this(hi, &path, NULL, &val);
> @@ -96,12 +96,8 @@ index_revision_adds(int *count, svn_fs_t
>        if ((change->change_kind == svn_fs_path_change_add)
>            || (change->change_kind == svn_fs_path_change_replace))
>          {
> -          const char *copyfrom_path;
> -          svn_revnum_t copyfrom_rev;
> -
> -          SVN_ERR(svn_fs_copied_from(&copyfrom_rev, &copyfrom_path,
> -                                     root, path, subpool));
> -          if (! (copyfrom_path && SVN_IS_VALID_REVNUM(copyfrom_rev)))
> +          if (! (change->copyfrom_path

The doc string for that parameter says:

  /** Copyfrom revision and path; this is only valid if copyfrom_known
   * is true. */