Changeset 13658 for trunk/ppStats/src/ppStats.c

Timestamp:

Jun 5, 2007, 3:08:13 PM (19 years ago)

Author:

Paul Price

Message:

Adding maskVal parameter to ppStats so that when it's called as a library function, it uses the nominated maskVal rather than it's own (which may be completely different from what the parent program may think it should mask).

File:

• trunk/ppStats/src/ppStats.c (modified) (2 diffs)

trunk/ppStats/src/ppStats.c

@@ -12 +12 @@
                     pmFPA *fpa,         // FPA for which to get statistics
                     pmFPAview *view,    // View for analysis
+                    psMaskType maskVal, // Value to mask
                     pmConfig *config    // Configuration
     )
 {
-    psExit status;
+    PS_ASSERT_PTR_NON_NULL(fpa, NULL)
+    PS_ASSERT_PTR_NON_NULL(view, NULL);
+    PS_ASSERT_PTR_NON_NULL(config, NULL);
+
+    ppStatsData *data = ppStatsDataAlloc(); // All the input data
 
     // Get the options, open the files
-    ppStatsData *data = ppStatsSetupFromRecipe(NULL, config);
+    if (!ppStatsSetupFromRecipe(data, config)) {
+        psError(PS_ERR_UNKNOWN, false, "Unable to get ppStats options from recipe.");
+        psFree(data);
+        return NULL;
+    }
+
+    // Override recipe mask value if one is provided
+    if (maskVal != 0) {
+        data->maskVal = maskVal;
+    }
 
     if (data->fpa) {
@@ -31 +45 @@
 
     // Go through the FPA and do the hard work
+    psExit status;                      // Status of statistics loop
     psMetadata *result = ppStatsLoop(&status, out, data, config);
     if (status != PS_EXIT_SUCCESS) {
-        psError (PS_ERR_UNKNOWN, false, "trouble getting stats\n");
+        psError (PS_ERR_UNKNOWN, false, "Not able to measure FPA statistics.\n");
         psFree(result);
-        psFree(data);
-        return (NULL);
+        psFree(data);
+        return (NULL);
     }