Author: kotkov
Date: Wed Nov 22 15:17:52 2017
New Revision: 1816061

URL: http://svn.apache.org/viewvc?rev=1816061&view=rev
Log:
Following up on r1815799, remove an excessive check for null relpath
in the maybe_set_lock_token_header().

Among the calling sites of this function, the relpath might be null only
within a proppatch_context_t (all other contexts disallow null relpaths),
and the corresponding calling site in setup_proppatch_headers() already
contains an explicit check for null.

* subversion/libsvn_ra_serf/commit.c
  (maybe_set_lock_token_header): Don't check for null relpath.

Modified:
    subversion/trunk/subversion/libsvn_ra_serf/commit.c

Modified: subversion/trunk/subversion/libsvn_ra_serf/commit.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/commit.c?rev=1816061&r1=1816060&r2=1816061&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/commit.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/commit.c Wed Nov 22 15:17:52 2017
@@ -687,7 +687,7 @@ maybe_set_lock_token_header(serf_bucket_
 {
   const char *token;
 
-  if (! (relpath && commit_ctx->lock_tokens))
+  if (! commit_ctx->lock_tokens)
     return SVN_NO_ERROR;
 
   if (! svn_hash_gets(commit_ctx->deleted_entries, relpath))


Reply via email to