subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <>
Subject [PATCH] svn/conflict-callbacks.c indentation fix
Date Fri, 02 Nov 2018 09:39:47 GMT
There are two gcc 8.x (I'm using 8.2.1) warnings from which catch those:

  if (foo) 

type of errors.  I fixed one case which looks obviously like a false 
positive in r1845556, but I'm not sure about the other one, in this 

The warning is:

subversion/svn/conflict-callbacks.c: In function ‘build_tree_conflict_options’:
subversion/svn/conflict-callbacks.c:1531:7: warning: this ‘if’ clause does not guard...
       if (all_options_are_dumb != NULL &&
subversion/svn/conflict-callbacks.c:1537:9: note: ...this statement, but the latter is misleadingly
indented as if it were guarded by the ‘if’
         if (*possible_moved_to_repos_relpaths == NULL)

Can someone familiar with the code check whether it's OK?

* subversion/svn/conflict-callbacks.c
  (build_tree_conflict_options): No functional change; fix
  indentation to avoid gcc -Wmisleading-indentation warning.

View raw message