Changeset 23989 for trunk/ppStats/src

Timestamp:

Apr 28, 2009, 11:21:56 AM (17 years ago)

Author:

eugene

Message:

always handle psVectorStats false return status (is an error); check for NAN results and handle as possible

Location:

trunk/ppStats/src

Files:

• ppStatsFromMetadataStats.c (modified) (1 diff)
• ppStatsReadout.c (modified) (1 diff)

trunk/ppStats/src/ppStatsFromMetadataStats.c

@@ -38 +38 @@
 
         psStats *stats = psStatsAlloc(option);
-        psVectorStats(stats, entry->vector, NULL, NULL, 0);
+        if (!psVectorStats(stats, entry->vector, NULL, NULL, 0)) {
+            psError(PS_ERR_UNKNOWN, false, "failure to measure stats for %s", entry->statistic);
+            continue;
+        }
 
         double value;

trunk/ppStats/src/ppStatsReadout.c

@@ -92 +92 @@
         }
         if (!psVectorStats(data->stats, sampleValues, NULL, sampleMask, 1)) {
-            psWarning("Unable to perform statistics on readout %s.\n", readoutName);
-            psErrorClear();
-        }
+            psError(PS_ERR_UNKNOWN, false, "failure to measure stats for readout %s", readoutName);
+        }
+        // XXX raise if we get a NAN? psWarning("Unable to perform statistics on readout %s.\n", readoutName);
         psFree(sampleValues);
         psFree(sampleMask);