From af155fcfb34a4dfb45ec05a2461f6fac3b2364aa Mon Sep 17 00:00:00 2001
From: Quincey Koziol <koziol@hdfgroup.org>
Date: Fri, 11 Jun 2004 09:08:04 -0500
Subject: [svn-r8657] Purpose:     Code optimization

Description:
    Eliminate redundant memory allocation for type conversion of variable-length
sequences.

Platforms tested:
    Solaris 2.7 (arabica)
    FreeBSD 4.10 (sleipnir) w/parallel
    Too minor to require h5committest
---
 src/H5Tvlen.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/H5Tvlen.c b/src/H5Tvlen.c
index 739c13c..d78f4d3 100644
--- a/src/H5Tvlen.c
+++ b/src/H5Tvlen.c
@@ -551,12 +551,13 @@ H5T_vlen_str_mem_read(H5F_t UNUSED *f, hid_t UNUSED dxpl_id, void *_vl, void *bu
 
     FUNC_ENTER_NOAPI_NOINIT_NOFUNC(H5T_vlen_str_mem_read)
 
-    /* check parameters */
-    assert(s);
-    assert(buf);
+    if(len>0) {
+        /* check parameters */
+        assert(s);
+        assert(buf);
 
-    if(len>0)
         HDmemcpy(buf,s,len);
+    } /* end if */
 
     FUNC_LEAVE_NOAPI(SUCCEED)
 }   /* end H5T_vlen_str_mem_read() */
@@ -796,7 +797,7 @@ H5T_vlen_disk_write(H5F_t *f, hid_t dxpl_id, void *_vl, void *buf, void *_bg, hs
 
     /* check parameters */
     assert(vl);
-    assert(buf);
+    assert(seq_len==0 || buf);
     assert(f);
     
     /* Free heap object for old data.  */
-- 
cgit v0.12