Changeset 7316 for trunk/ippTools/src/dettool.c

Timestamp:

Jun 2, 2006, 3:19:00 PM (20 years ago)

Author:

jhoblitt

Message:

change -normalized[frame] to require option -det_id

File:

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

trunk/ippTools/src/dettool.c

@@ -1174 +1174 @@
    
     // det_id is required
+    // psString det_id is unused... it is only checked for to make sure it is
+    // in config->where
+    bool status = false;
+    psString det_id = psMetadataLookupStr(&status, config->args, "-det_id");
+    if (!status) {
+        psError(PS_ERR_UNKNOWN, false, "failed to lookup value for -det_id");
+        return false;
+    }
+    if (!det_id) {
+        psError(PS_ERR_UNKNOWN, true, "-det_id is required");
+        return false;
+    }
 
     psArray *masterImfiles =
@@ -1205 +1217 @@
     PS_ASSERT_PTR_NON_NULL(config, false);
     
-    // det_id is required
+    // XXX det_id is requried as a simplification
     bool status = false;
     psString det_id = psMetadataLookupStr(&status, config->args, "-det_id");
@@ -1212 +1224 @@
         return false;
     }
+    if (!det_id) {
+        psError(PS_ERR_UNKNOWN, true, "-det_id is required");
+        return false;
+    }
 
     // find all detNormalizedIImfile for det_id
-    // XXX det_id is requried as a simplification
     psArray *masterImfiles =
         detNormalizedImfileSelectRowObjects(config->dbh, config->where, 0);