svn commit: r1102065 - /subversion/trunk/subversion/libsvn_ra_serf/commit.c

[julianfoad]

Author: julianfoad
Date: Wed May 11 20:17:07 2011
New Revision: 1102065

URL: http://svn.apache.org/viewvc?rev=1102065&view=rev
Log:
* /home/julianfoad/src/subversion-d/subversion/libsvn_ra_serf/commit.c
  (derive_old_val): Lose an unnecessary deep copy.

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=1102065&r1=1102064&r2=1102065&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/commit.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/commit.c Wed May 11 20:17:07 2011
@@ -675,7 +675,7 @@ typedef struct walker_baton_t {
  * then set *HAVE_OLD_VAL to TRUE and set *OLD_VAL_P to that old value
  * (which may be NULL); else set *HAVE_OLD_VAL to FALSE.
  *
- * Allocate *OLD_VAL_P in POOL. */
+ * The string pointed to by *OLD_VAL_P is not copied into POOL. */
 static svn_error_t *
 derive_old_val(svn_boolean_t *have_old_val,
                const svn_string_t **old_val_p,
@@ -694,7 +694,7 @@ derive_old_val(svn_boolean_t *have_old_v
       if (val)
         {
           *have_old_val = TRUE;
-          *old_val_p = svn_string_dup(val, pool);
+          *old_val_p = val;
         }
     }