Goto label warning with gcc -Wall

When not compiling with --enable-maintainer-mode, then you get this warning
with gcc -Wall:

subversion/mod_dav_svn/repos.c: In function `dav_svn_get_resource':
subversion/mod_dav_svn/repos.c:1030: warning: label `unknown_URI' defined but not used

The label is used here:

#ifdef SVN_DEBUG
  if (comb->res.type == DAV_RESOURCE_TYPE_UNKNOWN)
    {
      DBG0("DESIGN FAILURE: should not be UNKNOWN at this point");
      goto unknown_URI;
    }
#endif

and the goto is this:

 unknown_URI:
  /* Unknown URI. Return NULL to indicate "no resource" */
  *resource = NULL;
  return NULL;
}

Can the label code just be moved into the if statement, or is there a
reason that this code is split up? -1 on style points here. If not,
then let's put #ifdef SVN_DEBUG around the label and the code following it.

Best,
Blair

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:37:06 2006