Changeset 7278 for trunk/psModules/src/camera/pmHDUUtils.c

Timestamp:

Jun 1, 2006, 2:55:23 PM (20 years ago)

Author:

Paul Price

Message:

Addition of a vast quantity of assertions in public functions. Adopted a policy of using assert() within file-static functions (since they are only called internally, any errors there are problems with the program) and using the PS_ASSERT_WHATEVER() macros within public functions. Cleaned a few things up in the process.

File:

• trunk/psModules/src/camera/pmHDUUtils.c (modified) (5 diffs)

trunk/psModules/src/camera/pmHDUUtils.c

@@ -8 +8 @@
                    )
 {
+    PS_ASSERT_PTR_NON_NULL(fpa, NULL);
     return fpa->hdu;
 }
@@ -14 +15 @@
                     )
 {
+    PS_ASSERT_PTR_NON_NULL(chip, NULL);
+
     pmHDU *hdu = chip->hdu;             // The HDU information
     if (!hdu) {
@@ -25 +28 @@
                     )
 {
+    PS_ASSERT_PTR_NON_NULL(cell, NULL);
+
     pmHDU *hdu = cell->hdu;             // The HDU information
     if (!hdu) {
@@ -33 +38 @@
 }
 
-pmHDU *pmHDUFromReadout (pmReadout *readout)
+pmHDU *pmHDUFromReadout(pmReadout *readout)
 {
+    PS_ASSERT_PTR_NON_NULL(readout, NULL);
 
     pmCell *cell = readout->parent; // cell containing this readout;
-    pmHDU *hdu = pmHDUFromCell (cell);
+    pmHDU *hdu = pmHDUFromCell(cell);
     return hdu;
 }
@@ -64 +70 @@
                )
 {
+    PS_ASSERT_PTR_NON_NULL(hdu,);
+
     if (hdu->phu) {
         psTrace(__func__, level, "HDU: %s (PHU)\n", hdu->extname);