You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Philip Martin <ph...@wandisco.com> on 2010/05/05 08:45:06 UTC
Re: svn commit: r941094 - /subversion/trunk/subversion/libsvn_client/commit_util.c
neels@apache.org writes:
> Author: neels
> Date: Tue May 4 23:31:42 2010
> New Revision: 941094
> URL: http://svn.apache.org/viewvc?rev=941094&view=rev
> Log:
> One less use of entry_t.
> if (is_added)
> {
> - SVN_ERR(svn_wc__node_get_copyfrom_info(&entry_copyfrom_url,
> - &entry_copyfrom_rev,
> + svn_boolean_t is_copy_target;
> + const char *copyfrom_url;
> + svn_revnum_t copyfrom_rev;
> +
> + SVN_ERR(svn_wc__node_get_copyfrom_info(©from_url,
> + ©from_rev,
> &is_copy_target,
> ctx->wc_ctx, local_abspath,
> scratch_pool, scratch_pool));
../src/subversion/libsvn_client/commit_util.c:565: warning: declaration of ‘copyfrom_url’ shadows a parameter
../src/subversion/libsvn_client/commit_util.c:367: warning: shadowed declaration is here
--
Philip
Re: svn commit: r941094 - /subversion/trunk/subversion/libsvn_client/commit_util.c
Posted by Neels J Hofmeyr <ne...@elego.de>.
On 05/05/2010 10:45 AM, Philip Martin wrote:
> neels@apache.org writes:
>
>> Author: neels
>> Date: Tue May 4 23:31:42 2010
>> New Revision: 941094
>> URL: http://svn.apache.org/viewvc?rev=941094&view=rev
>> Log:
>> One less use of entry_t.
>
>> if (is_added)
>> {
>> - SVN_ERR(svn_wc__node_get_copyfrom_info(&entry_copyfrom_url,
>> - &entry_copyfrom_rev,
>> + svn_boolean_t is_copy_target;
>> + const char *copyfrom_url;
>> + svn_revnum_t copyfrom_rev;
>> +
>> + SVN_ERR(svn_wc__node_get_copyfrom_info(©from_url,
>> + ©from_rev,
>> &is_copy_target,
>> ctx->wc_ctx, local_abspath,
>> scratch_pool, scratch_pool));
>
> ../src/subversion/libsvn_client/commit_util.c:565: warning: declaration of ‘copyfrom_url’ shadows a parameter
> ../src/subversion/libsvn_client/commit_util.c:367: warning: shadowed declaration is here
oops, thx!
I want to fix that whole duplicate-get_copyfrom_info thing anyway
(later/soon). That var shadowing shouldn't hurt for now, though, except for
the warning.
~Neels