Re: svn commit: r40213 - trunk/subversion/libsvn_client

On Fri, Oct 23, 2009 at 23:13, Paul T. Burba <pburba_at_collab.net> wrote:
>...
> +++ trunk/subversion/libsvn_client/merge.c      Fri Oct 23 20:13:32 2009        (r40213)
>...
> @@ -4288,11 +4286,11 @@ calculate_merge_inheritance(apr_array_he
>  }
>
>  /* Calculate the new mergeinfo for the target tree based on the merge
> -   info for TARGET_WCPATH and MERGES (a mapping of WC paths to range
> +   info for TARGET_ABSPATH and MERGES (a mapping of WC paths to range
>    lists), and record it in the WC (at, and possibly below,
> -   TARGET_WCPATH). */
> +   TARGET_ABSPATH). */
>  static svn_error_t *
> -update_wc_mergeinfo(const char *target_wcpath,
> +update_wc_mergeinfo(const char *target_abspath,
>                     const char *repos_rel_path,
>                     apr_hash_t *merges,
>                     svn_boolean_t is_rollback,
> @@ -4358,7 +4356,7 @@ update_wc_mergeinfo(const char *target_w
>
>       /* ASSUMPTION: "target_wcpath" is always both a parent and
>          prefix of "path". */
> -      len = strlen(target_wcpath);
> +      len = strlen(target_abspath);
>       if (len < strlen(path))

This is looking a bit hinky. The TARGET_ABSPATH parameter is used just
*once* in this function? And all that is done with it, is to get the
length? What?

>...

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2411053
Received on 2009-10-24 17:48:01 CEST