Changeset 42777

Timestamp:

Dec 29, 2024, 4:20:40 PM (19 months ago)

Author:

eugene

Message:

change assert to an error

File:

• branches/eam_branches/ipp-ppmerge-20241229/psModules/src/camera/pmFPAConstruct.c (modified) (2 diffs)

branches/eam_branches/ipp-ppmerge-20241229/psModules/src/camera/pmFPAConstruct.c

@@ -52 +52 @@
 
 // Parse a list of first:second:third pairs in a string
+// EAM : this function takes an input 'string' and attempts to parse out
+// groups of the form FIRST:SECOND:THIRD into the arrays (first, second, third).
+
+// The input string may have multiple entries of this form separated by spaces, commas, or
+// semi-colons.  The number of arrays which are supplied must match the string format or
+// an error will be raised.
+
+// e.g., the string could be CHIP:CELL:TYPE in which case all three arrays must exist
+// or, the string could be CHIP:CELL only, in which case the array 'third' must be NULL
+ 
 static int parseContent(psArray **first, // Array of the first values
                          psArray **second, // Array of the second values
@@ -83 +93 @@
     while ((value = psListGetAndIncrement(valuesIter))) {
         psArray *fst = psStringSplitArray(value, ":", true); // First, second, third
-        assert (numArrays == fst->n);
+        if (fst->n != numArrays) {
+          psError(PS_ERR_BAD_PARAMETER_VALUE, false, "string %s does not match expected format (%ld colon-separated items supplied, %d expected)", value, fst->n, numArrays);
+          return 0;
+        }
         switch (numArrays) {
           case 3: