Changeset 3813 for trunk/pois/src/poisRejectStamps.c

Timestamp:

Apr 29, 2005, 10:53:49 AM (21 years ago)

Author:

rhl

Message:

Make poisRejectStamps return a list of rejected stamps

File:

• trunk/pois/src/poisRejectStamps.c (modified) (6 diffs)

trunk/pois/src/poisRejectStamps.c

@@ -6 +6 @@
 #define SQUARE(x) ((x)*(x))
 
-bool poisRejectStamps(psArray *stamps,  // Array of stamps
-                      psImage *mask,    // Mask image
-                      const psVector *deviations, // Vector of deviations for the stamps
-                      const poisConfig *config // Configuration
+psList *restrict poisRejectStamps(psArray *stamps, // Array of stamps
+                                  psImage *mask,        // Mask image
+                                  const psVector *deviations, // Vector of deviations for the stamps
+                                  const poisConfig *config // Configuration
     )
 {
@@ -21 +21 @@
     assert(deviations->type.type == PS_TYPE_F32);
 
-    bool masked = false;                // Have we masked any stamps?
     psVector *devMask = psVectorAlloc(stamps->n, PS_TYPE_U8); // Mask for statistics
 
@@ -30 +29 @@
         poisStamp *stamp = stamps->data[i];     // The stamp
         // Only interested in the stamps that we're actually using
-        if (stamp->status == POIS_STAMP_USED) {
+        if (stamp != NULL && stamp->status == POIS_STAMP_USED) {
             meanDev += SQUARE(deviations->data.F32[i]);
             numDev++;
@@ -41 +40 @@
 
     // Reject stamps
+    psList *rejected = NULL;            // rejected stamps
     for (int s = 0; s < deviations->n; s++) {
         poisStamp *stamp = stamps->data[s];
+        if (stamp == NULL) {
+            continue;
+        }
+        
         if (stamp->status == POIS_STAMP_USED && fabsf(deviations->data.F32[s]) > limit) {
-            masked = true;
             psTrace("pois.rejectStamps", 1, "Rejecting stamp %d (%d,%d): %f\n", s, stamp->x, stamp->y,
                     deviations->data.F32[s]);
@@ -56 +59 @@
             
             // Mark stamp for replacement
-            stamp->status = POIS_STAMP_RESET;
+            stamps->data[s] = NULL;
+            
+            if (rejected == NULL) {
+                rejected = psListAlloc(stamp);
+            } else {
+                bool failed = psListAdd(rejected, PS_LIST_TAIL, stamp);
+                assert(!failed);
+            }
+            psFree(stamp);
         } // Bad stamps
     } // Iterating over stamps
@@ -62 +73 @@
     psFree(devMask);
 
-    return masked;
+    return rejected;
 }