diff common.cpp @ 370:2d5c3f8e8c22

Merge gcc-4.3-cleanups branch (-r629:642, but I consider that branch as having served its purpose) onto the trunk. Now compiles cleanly even with a fairly picky gcc. I await version 4.4 with bated breath.
author mas01cr
date Wed, 12 Nov 2008 15:40:40 +0000
parents 94c18f128ce8
children 4e68f7d4d524 342822c2d49a fbcc1303dfeb
line wrap: on
line diff
--- a/common.cpp	Wed Nov 12 10:21:06 2008 +0000
+++ b/common.cpp	Wed Nov 12 15:40:40 2008 +0000
@@ -53,15 +53,15 @@
  
 
     if(isServer) {
-    /* FIXME: I think this is leaky -- we never delete err.  actually
-       deleting it is tricky, though; it gets placed into some
-       soap-internal struct with uncertain extent... -- CSR,
-       2007-10-01 */
-    char *err = new char[256]; /* FIXME: overflows */
-    snprintf(err, 255, "%s: %s\n%s", a, b, sysFunc ? strerror(errno) : "");
-    /* FIXME: actually we could usefully do with a properly structured
-       type, so that we can throw separate faultstring and details.
-       -- CSR, 2007-10-01 */
+        /* FIXME: I think this is leaky -- we never delete err.
+           actually deleting it is tricky, though; it gets placed into
+           some soap-internal struct with uncertain extent... -- CSR,
+           2007-10-01 */
+        char *err = new char[256]; /* FIXME: overflows */
+        snprintf(err, 255, "%s: %s\n%s", a, b, sysFunc ? strerror(errno) : "");
+        /* FIXME: actually we could usefully do with a properly
+           structured type, so that we can throw separate faultstring
+           and details.  -- CSR, 2007-10-01 */
         throw(err);
     } else if (UseApiError){
         apierrortemp=-1;