Changeset 39028 for trunk/ippTools/src/addtool.c

Timestamp:

Nov 2, 2015, 2:57:52 PM (11 years ago)

Author:

eugene

Message:

add stack_id as part of output from fftool -summary; do not call test macros for items used only in where statements

File:

• trunk/ippTools/src/addtool.c (modified) (3 diffs)

trunk/ippTools/src/addtool.c

@@ -421 +421 @@
       psStringAppend(&query, " GROUP BY ff_id ");  //needs to be checked, but should be fine: want 1 ff summary cmf.
     }
-    psError(PS_ERR_UNKNOWN, false , query);
+    psError(PS_ERR_UNKNOWN, false, "query: %s\n", query);
 
     psTrace("addtool.c", PS_LOG_INFO,"query: \n\n%s\n\n",query);
@@ -1199 +1199 @@
     psMetadata *where = psMetadataAlloc();
     if (multiadd) {
-      //required, check that they are there
-      PXOPT_LOOKUP_STR(multiaddlabel, config->args, "-multiaddlabel",true,false);
-      PXOPT_LOOKUP_STR(stage, config->args, "-stage",true,false);
-      PXOPT_LOOKUP_S64(stage_id, config->args, "-stage_id",true,false);
-      //do the where
+      // NOTE: these tests are not needed because the PXOPT_COPY_* calls below return FALSE on failure to find the item
+      // required, check that they are there
+      // PXOPT_LOOKUP_STR(multiaddlabel, config->args, "-multiaddlabel", true, false);
+      // PXOPT_LOOKUP_STR(stage, config->args, "-stage", true, false);
+      // PXOPT_LOOKUP_S64(stage_id, config->args, "-stage_id", true, false);
+
+      // do the where
       PXOPT_COPY_STR(config->args, where, "-multiaddlabel",   "addRun.label",   "==");
       PXOPT_COPY_STR(config->args, where, "-stage", "addRun.stage","==");
@@ -1211 +1213 @@
       //if not multiadd mode, then do it by add_id
       // required, check for add_id
-      PXOPT_LOOKUP_S64(add_id, config->args, "-add_id",true,false);
+      // PXOPT_LOOKUP_S64(add_id, config->args, "-add_id",true,false);
       PXOPT_COPY_S64(config->args, where, "-add_id",   "addRun.add_id",   "==");