Author: brane
Date: Sat Apr 27 12:28:44 2013
New Revision: 1476563
URL: http://svn.apache.org/r1476563
Log:
* subversion/svnserve/serve.c (update): Silence a warning about conversion
of apr_uint64_t to svn_boolean_t by explicitly making the argument to
svn_log__update a boolean expression.
(update, switch_cmd): Incidentally, replace the cases where we had
checks like this: "variable == TRUE" to "variable != FALSE" instead,
since the latter is more robust given C's truth-value rules.
Modified:
subversion/trunk/subversion/svnserve/serve.c
Modified: subversion/trunk/subversion/svnserve/serve.c
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/svnserve/serve.c?rev=1476563&r1=1476562&r2=1476563&view=diff
==============================================================================
--- subversion/trunk/subversion/svnserve/serve.c (original)
+++ subversion/trunk/subversion/svnserve/serve.c Sat Apr 27 12:28:44 2013
@@ -1843,8 +1843,8 @@ static svn_error_t *update(svn_ra_svn_co
SVN_ERR(accept_report(&is_checkout, NULL,
conn, pool, b, rev, target, NULL, TRUE,
depth,
- (send_copyfrom_args == TRUE) /* send_copyfrom_args */,
- (ignore_ancestry == TRUE) /* ignore_ancestry */));
+ (send_copyfrom_args != FALSE), /* send_copyfrom_args */
+ (ignore_ancestry != FALSE))); /* ignore_ancestry */
if (is_checkout)
{
SVN_ERR(log_command(b, conn, pool, "%s",
@@ -1855,7 +1855,8 @@ static svn_error_t *update(svn_ra_svn_co
{
SVN_ERR(log_command(b, conn, pool, "%s",
svn_log__update(full_path, rev, depth,
- send_copyfrom_args, pool)));
+ (send_copyfrom_args != FALSE),
+ pool)));
}
return SVN_NO_ERROR;
@@ -1905,7 +1906,7 @@ static svn_error_t *switch_cmd(svn_ra_sv
return accept_report(NULL, NULL,
conn, pool, b, rev, target, switch_path, TRUE,
depth,
- (send_copyfrom_args == TRUE) /* send_copyfrom_args */,
+ (send_copyfrom_args != FALSE) /* send_copyfrom_args */,
(ignore_ancestry != FALSE) /* ignore_ancestry */);
}
