Changeset 8598

Timestamp:

Aug 25, 2006, 11:50:07 AM (20 years ago)

Author:

jhoblitt

Message:

add psStringAppendV()
change psStringAppend() to be a wrapper around psStringAppendV()

Location:

trunk/psLib/src/sys

Files:

• psString.c (modified) (4 diffs)
• psString.h (modified) (3 diffs)

trunk/psLib/src/sys/psString.c

@@ -13 +13 @@
  *  @author David Robbins, MHPCC
  *
- *  @version $Revision: 1.41 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2006-08-18 01:59:13 $
+ *  @version $Revision: 1.42 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2006-08-25 21:50:07 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -98 +98 @@
                        ...)
 {
+    va_list ap;
+    va_start(ap, format);
+    ssize_t length = psStringAppendV(dest, format, ap);
+    va_end(ap);
+
+    return length;
+}
+
+ssize_t psStringAppendV(char **dest,
+                        const char *format,
+                        va_list ap)
+{
     PS_ASSERT_PTR_NON_NULL(format, 0);
-    va_list         args;
     size_t          length;             // complete string length (sans \0)
     size_t          oldLength;          // original string length (sans \0)
@@ -115 +126 @@
 
     // find the size of the string to append
-    va_start(args, format);
+    va_list         apCopy;
     // C99 guarentees vsnprintf() to work as expected with size = 0
-    tailLength = vsnprintf(*dest, 0, format, args);
-    va_end(args);
+    va_copy(apCopy, ap);
+    tailLength = vsnprintf(*dest, 0, format, apCopy);
+    va_end(apCopy);
 
     // if the new tail is zero length, return the length of the old string.  if
@@ -134 +146 @@
 
     // append tail + \0
-    va_start(args, format);
-    vsnprintf(*dest + oldLength, tailLength + 1, format, args);
-    va_end(args);
+    // XXX this second call to va_copy() isn't strictly nessicary as the
+    // calling function can't assume we won't modify the va_list.  However, we
+    // have decided to error on the side of caution.
+    va_copy(apCopy, ap);
+    vsnprintf(*dest + oldLength, tailLength + 1, format, apCopy);
+    va_end(apCopy);
 
     return length;

trunk/psLib/src/sys/psString.h

@@ -14 +14 @@
  *  @author David Robbins, MHPCC
  *
- *  @version $Revision: 1.26 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2006-07-14 02:26:25 $
+ *  @version $Revision: 1.27 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2006-08-25 21:49:47 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -24 +24 @@
 
 #include <sys/types.h>
+#include <stdarg.h>
 #include "psType.h"
 #include "psList.h"
@@ -100 +101 @@
 );
 
+/** Appends a format onto a string
+ *
+ * This function shall allocate a new string if dest is NULL.  dest shall be
+ * automatically extended to the size of the new string.
+ *
+ * @return ssize_t:     The length of the new string (excluding '\0')
+ */
+ssize_t psStringAppendV(
+    char **dest,                        ///< existing string
+    const char *format,                 ///< format to append
+    va_list ap                          ///< va_list of format arguments
+);
+
 /** Prepends a format onto a string
  *