Changeset 18336 for trunk/ippTools/src/pxregister.c

Timestamp:

Jun 26, 2008, 3:39:54 PM (18 years ago)

Author:

jhoblitt

Message:

consitently handle *_id fields as numeric values instead of converting back of forth between ints and strings

File:

• trunk/ippTools/src/pxregister.c (modified) (2 diffs)

trunk/ippTools/src/pxregister.c

@@ -29 +29 @@
 #include "pxregister.h"
 
-bool pxnewExpSetState(pxConfig *config, const char *exp_id, const char *state)
+bool pxnewExpSetState(pxConfig *config, psS64 exp_id, const char *state)
 {
-    PS_ASSERT_PTR_NON_NULL(exp_id, false);
+    if (!exp_id) {
+        psError(PS_ERR_UNKNOWN, true, "0 is not a valid exp_id");
+        return false;
+    }
     PS_ASSERT_PTR_NON_NULL(state, false);
 
@@ -46 +49 @@
     }
 
-    char *query = "UPDATE newExp SET state = '%s' WHERE exp_id = '%s'";
+    char *query = "UPDATE newExp SET state = '%s' WHERE exp_id = %"PRId64;
     if (!p_psDBRunQuery(config->dbh, query, state, exp_id)) {
         psError(PS_ERR_UNKNOWN, false,
-                "failed to change state for exp_id %s", exp_id);
+                "failed to change state for exp_id %"PRId64, exp_id);
         return false;
     }