Changeset 30222

Timestamp:

Jan 7, 2011, 2:20:29 PM (15 years ago)

Author:

eugene

Message:

add an assert to trigger on duplicate peaks, incase this is a possibility

File:

• branches/eam_branches/ipp-20101205/psModules/src/objects/pmFootprintAssignPeaks.c (modified) (1 diff)

branches/eam_branches/ipp-20101205/psModules/src/objects/pmFootprintAssignPeaks.c

@@ -88 +88 @@
         fp->peaks = psArraySort(fp->peaks, pmPeakSortBySN);
 
-        // XXX WHY am I culling duplicates?  how can there be duplicates?
+        // XXX check for an assert on duplicates (I don't think they can happen, but
+        // let's double check for now)
+
+        for (int j = 1; j < fp->peaks->n; j++) {
+            psAssert (fp->peaks->data[j] != fp->peaks->data[j-1], "duplicate peak!");
+        }
         continue;
 
+        // XXX WHY am I culling duplicates?  how can there be duplicates?
         // XXX EAM : the algorithm below should be much faster than using psArrayRemove if
         // the number of peaks in the footprint is large, or if there are no duplicates.