Changeset 1497 for trunk/src

Timestamp:

02/04/09 17:42:20 (15 years ago)

Author:

Max Voronkov

Message:

fixed #149. The problem was inside getMask method. In some rare circumstances largely when single channel detections are allowed and with spectral averaging enabled, the method failed to flag the lines which have already been detected and went into infinite loop by detecting them again. The condition of leaving the loop (no new lines found) was never satisfied as a result of this.

File:

• trunk/src/STLineFinder.cpp (modified) (1 diff)

trunk/src/STLineFinder.cpp

@@ -870 +870 @@
        // iterator through lines
        std::list<std::pair<int,int> >::const_iterator cli=lines.begin();
-       for (int ch=0;ch<int(res_mask.size());++ch)
+       for (int ch=0;ch<int(res_mask.size());++ch) {
             if (ch<edge.first || ch>=edge.second) res_mask[ch]=false;
             else if (!mask[ch]) res_mask[ch]=false;
             else {
                     res_mask[ch]=!invert; // no line by default
-                    if (cli==lines.end()) continue;
-                    if (ch>=cli->first && ch<cli->second)
-                        res_mask[ch]=invert; // this is a line
-                    if (ch>=cli->second)
-                        ++cli; // next line in the list
-                 }
-
+                    if (cli!=lines.end())
+                        if (ch>=cli->first && ch<cli->second)
+                             res_mask[ch]=invert; // this is a line
+            }
+            if (cli!=lines.end())
+                if (ch>=cli->second) {
+                    ++cli; // next line in the list
+                }
+       }
        return res_mask;
   }