You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Branko Čibej <br...@wandisco.com> on 2015/02/06 11:06:44 UTC

Re: svn commit: r1657756 - /subversion/trunk/subversion/libsvn_wc/wc_db_update_move.c

On 06.02.2015 09:40, stefan2@apache.org wrote:
> Author: stefan2
> Date: Fri Feb  6 08:40:35 2015
> New Revision: 1657756
>
> URL: http://svn.apache.org/r1657756
> Log:
> Fix "variable 'shadowed' set but not used" compiler warning.
> This seems to be a fallout of r1657725.
>
> * subversion/libsvn_wc/wc_db_update_move.c
>   (tc_editor_add_directory): Remove write-only variable IS_CONFLICT.

The variable's name is 'is_conflicted', not 'shadowed' and not
'IS_CONFLICT'.

-- Brane

Re: svn commit: r1657756 - /subversion/trunk/subversion/libsvn_wc/wc_db_update_move.c

Posted by Stefan Fuhrmann <st...@wandisco.com>.

On Fri, Feb 6, 2015 at 11:06 AM, Branko Čibej <br...@wandisco.com> wrote:

> On 06.02.2015 09:40, stefan2@apache.org wrote:
> > Author: stefan2
> > Date: Fri Feb  6 08:40:35 2015
> > New Revision: 1657756
> >
> > URL: http://svn.apache.org/r1657756
> > Log:
> > Fix "variable 'shadowed' set but not used" compiler warning.
> > This seems to be a fallout of r1657725.
> >
> > * subversion/libsvn_wc/wc_db_update_move.c
> >   (tc_editor_add_directory): Remove write-only variable IS_CONFLICT.
>
> The variable's name is 'is_conflicted', not 'shadowed' and not
> 'IS_CONFLICT'.
>

Oops. You are right, that's been a copy'n'pasto from wc_db.c.
Corrected now.

-- Stefan^2.