Changeset 12394 for trunk/psLib/src/db/psDB.c

Timestamp:

Mar 9, 2007, 3:24:40 PM (19 years ago)

Author:

jhoblitt

Message:

change psDBSelectColumn() to return NULL instead of "" for NULL database fields

File:

• trunk/psLib/src/db/psDB.c (modified) (3 diffs)

trunk/psLib/src/db/psDB.c

@@ -12 +12 @@
  *  @author Joshua Hoblitt
  *
- *  @version $Revision: 1.138 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2007-03-05 21:17:34 $
+ *  @version $Revision: 1.139 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2007-03-10 01:24:40 $
  *
  *  Copyright (C) 2005-2006  Joshua Hoblitt, University of Hawaii
@@ -416 +416 @@
         dataSize = *(mysql_fetch_lengths(result));
 
-        // represent NULL as an empty string
+        // represent NULL as a NULL string
         if (row[0] == NULL) {
-            data = psStringCopy("");
-        } else {
-            data = psAlloc(dataSize+1);
-            memcpy(data, row[0], dataSize);
-            ((char*)data)[dataSize] = '\0';
-        }
+            psArrayAdd(column, 0, NULL);
+            continue;
+        }
+
+        // make a copy of the data
+        data = psAlloc(dataSize+1);
+        memcpy(data, row[0], dataSize);
+        ((char*)data)[dataSize] = '\0';
 
         // add field to return array
@@ -441 +443 @@
 { \
     psPtr           myNaN; \
-    int             i; \
     \
-    for (i = 0; i < source->n; i++) { \
-        if (strlen(source->data[i])) { \
+    for (long i = 0; i < source->n; i++) { \
+        if (source->data[i]) { \
             dest[i] = (type)conv(source->data[i]); \
         } else { \