Changeset 7278 for trunk/psModules/src/camera/pmHDU.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/pmHDU.c (modified) (4 diffs)

trunk/psModules/src/camera/pmHDU.c

@@ -61 +61 @@
     psMemSetDeallocator(hdu, (psFreeFunc)hduFree);
 
-    if (! extname || strlen(extname) == 0) {
+    if (!extname || strlen(extname) == 0) {
         hdu->phu = true;
         hdu->extname = psStringCopy("PHU");
@@ -86 +86 @@
                     )
 {
-    assert(hdu);
-    assert(fits);
+    PS_ASSERT_PTR_NON_NULL(hdu, false);
+    PS_ASSERT_PTR_NON_NULL(fits, false);
 
     // Move to the appropriate extension
@@ -113 +113 @@
               )
 {
-    assert(hdu);
-    assert(fits);
+    PS_ASSERT_PTR_NON_NULL(hdu, false);
+    PS_ASSERT_PTR_NON_NULL(fits, false);
 
     // Read the header; includes the move
@@ -159 +159 @@
                )
 {
-    assert(hdu);
-    assert(fits);
+    PS_ASSERT_PTR_NON_NULL(hdu, false);
+    PS_ASSERT_PTR_NON_NULL(fits, false);
 
     psTrace(__func__, 7, "Writing HDU %s\n", hdu->extname);