On Tue, Nov 13, 2012 at 05:32:00PM -0000, cmpil...@apache.org wrote:
> Author: cmpilato
> Date: Tue Nov 13 17:31:59 2012
> New Revision: 1408849
> 
> URL: http://svn.apache.org/viewvc?rev=1408849&view=rev
> Log:
> * subversion/libsvn_ra_serf/update.c
>   (end_report): Remove redundant boolean assignment ("if (X) { X = TRUE }").

Are you sure this is not a bug checking the wrong X or setting the wrong
X to TRUE? I don't know ra_serf very well so this isn't obvious to me.

> @@ -1969,9 +1969,6 @@ end_report(svn_ra_serf__xml_parser_t *pa
>         */
>        if (info->dir->fetch_props)
>          {
> -          /* Unconditionally set fetch_props now. */
> -          info->dir->fetch_props = TRUE;
> -

Reply via email to