Changeset 13796

Timestamp:

Jun 13, 2007, 12:04:15 PM (19 years ago)

Author:

Paul Price

Message:

No HDU doesn't mean an error (if we have a FITS file), but it means we
simply weren't given the file that contains that HDU.

File:

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

trunk/ppStats/src/ppStatsLoop.c

@@ -85 +85 @@
         return PS_EXIT_SUCCESS;
     }
+    pmHDU *hdu = pmHDUFromCell(cell); // HDU for cell
+    if (!hdu || hdu->blankPHU) {
+        if (fits) {
+            // No HDU means there's no data in this cell
+            return PS_EXIT_SUCCESS;
+        } else {
+            psError PS_ERR_UNKNOWN, false, "Can't find HDU for cell\n");
+            return PS_EXIT_CONFIG_ERROR;
+        }
+    }
 
     // Cell-level results
@@ -115 +125 @@
         // Nothing further to do --- don't want to waste our time reading the data
         goto cellDone;
-    }
-
-    pmHDU *hdu = pmHDUFromCell(cell); // HDU for cell
-    if (!hdu || hdu->blankPHU) {
-        psError (PS_ERR_UNKNOWN, false, "trouble finding HDU for cell\n");
-        psFree(cellResults);
-        return PS_EXIT_CONFIG_ERROR;
     }