svn commit: r1684012 - in /subversion/trunk/subversion: libsvn_delta/text_delta.c mod_dav_svn/liveprops.c

[ivan]

Author: ivan
Date: Sun Jun  7 09:48:17 2015
New Revision: 1684012

URL: http://svn.apache.org/r1684012
Log:
Small code cleanup.

* subversion/libsvn_delta/text_delta.c
  (svn_txdelta_window_dup): Use apr_pmemdup() instead of apr_palloc()
   followed by memcpy().
* subversion/mod_dav_svn/liveprops.c
  (get_last_modified_time): Do not use memcpy to copy apr_time_t value.

Modified:
    subversion/trunk/subversion/libsvn_delta/text_delta.c
    subversion/trunk/subversion/mod_dav_svn/liveprops.c

Modified: subversion/trunk/subversion/libsvn_delta/text_delta.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_delta/text_delta.c?rev=1684012&r1=1684011&r2=1684012&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_delta/text_delta.c (original)
+++ subversion/trunk/subversion/libsvn_delta/text_delta.c Sun Jun  7 09:48:17 
2015
@@ -180,8 +180,7 @@ svn_txdelta_window_dup(const svn_txdelta
   build_baton.num_ops = window->num_ops;
   build_baton.src_ops = window->src_ops;
   build_baton.ops_size = window->num_ops;
-  build_baton.ops = apr_palloc(pool, ops_size);
-  memcpy(build_baton.ops, window->ops, ops_size);
+  build_baton.ops = apr_pmemdup(pool, window->ops, ops_size);
   build_baton.new_data =
     svn_stringbuf_create_from_string(window->new_data, pool);
 

Modified: subversion/trunk/subversion/mod_dav_svn/liveprops.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/liveprops.c?rev=1684012&r1=1684011&r2=1684012&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/liveprops.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/liveprops.c Sun Jun  7 09:48:17 2015
@@ -239,7 +239,7 @@ get_last_modified_time(const char **date
     }
 
   if (timeval)
-    memcpy(timeval, &timeval_tmp, sizeof(*timeval));
+    *timeval = timeval_tmp;
 
   if (! datestring)
     return 0;