You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by iv...@apache.org on 2015/10/31 20:08:59 UTC

svn commit: r1711672 - /subversion/trunk/subversion/libsvn_fs_fs/tree.c

Author: ivan
Date: Sat Oct 31 19:08:59 2015
New Revision: 1711672

URL: http://svn.apache.org/viewvc?rev=1711672&view=rev
Log:
Use more specific error code in FSFS implementation of svn_fs_contents_changed()
and svn_fs_contents_different().

Suggested by: rhuijben

* subversion/libsvn_fs_fs/tree.c
  (fs_contents_changed): Use SVN_ERR_FS_NOT_FILE instead of SVN_ERR_FS_GENERAL
   error code if one of provided path is not a file.

Modified:
    subversion/trunk/subversion/libsvn_fs_fs/tree.c

Modified: subversion/trunk/subversion/libsvn_fs_fs/tree.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/tree.c?rev=1711672&r1=1711671&r2=1711672&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/tree.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/tree.c Sat Oct 31 19:08:59 2015
@@ -3239,13 +3239,13 @@ fs_contents_changed(svn_boolean_t *chang
   /* Make sure that path is file. */
   if (svn_fs_fs__dag_node_kind(node1) != svn_node_file)
     return svn_error_createf
-      (SVN_ERR_FS_GENERAL, NULL, _("'%s' is not a file"), path1);
+      (SVN_ERR_FS_NOT_FILE, NULL, _("'%s' is not a file"), path1);
 
   SVN_ERR(get_dag(&node2, root2, path2, pool));
   /* Make sure that path is file. */
   if (svn_fs_fs__dag_node_kind(node2) != svn_node_file)
     return svn_error_createf
-      (SVN_ERR_FS_GENERAL, NULL, _("'%s' is not a file"), path2);
+      (SVN_ERR_FS_NOT_FILE, NULL, _("'%s' is not a file"), path2);
 
   return svn_fs_fs__dag_things_different(NULL, changed_p,
                                          node1, node2, strict, pool);