Update edge_drawing.cpp

Author: sturkmen72

modules/ximgproc/src/edge_drawing.cpp

@@ -2466,51 +2466,53 @@ void EdgeDrawingImpl::TestSegment(int i, int index1, int index2)
     TestSegment(i, start, index2);
 }
 
-//----------------------------------------------------------------------------------------------
-// After the validation of the edge segments, extracts the valid ones
-// In other words, updates the valid segments' pixel arrays and their lengths
+// After validating the edge segments, this function extracts the valid ones.
+// Specifically, it updates the valid segments' pixel arrays and their lengths based on validation.
 void EdgeDrawingImpl::ExtractNewSegments()
 {
     vector< vector<Point> > validSegments;
-    int noSegments = 0;
 
     for (int i = 0; i < segmentNos; i++) {
         int start = 0;
         while (start < (int)segmentPoints[i].size()) {
 
+            // Find the first valid start point
             while (start < (int)segmentPoints[i].size()) {
                 int r = segmentPoints[i][start].y;
                 int c = segmentPoints[i][start].x;
 
-                if (edgeImg[r * width + c]) break;
+                if (edgeImg[r * width + c]) break; // Found valid point
                 start++;
             }
 
             int end = start + 1;
+
+            // Find the end of the valid segment
             while (end < (int)segmentPoints[i].size()) {
                 int r = segmentPoints[i][end].y;
                 int c = segmentPoints[i][end].x;
 
-                if (edgeImg[r * width + c] == 0) break;
+                if (edgeImg[r * width + c] == 0) break; // End of segment
                 end++;
             }
 
+            // Length of the segment
             int len = end - start;
+
+            // Only accept segments of length >= 10
             if (len >= 10) {
-                // A new segment. Accepted only only long enough (whatever that means)
-                //segments[noSegments].pixels = &map->segments[i].pixels[start];
-                //segments[noSegments].noPixels = len;
-                validSegments.push_back(vector<Point>());
-                vector<Point> subVec(&segmentPoints[i][start], &segmentPoints[i][end]);
-                validSegments[noSegments] = subVec;
-                noSegments++;
+                vector<Point> subVec(segmentPoints[i].begin() + start, segmentPoints[i].begin() + end); // Safer bounds
+                validSegments.push_back(subVec); // Add to valid segments
             }
+
+            // Move start to next unprocessed point
             start = end + 1;
         }
     }
 
+    // Replace the old segments with valid segments
     segmentPoints = validSegments;
-    segmentNos = noSegments;
+    segmentNos = validSegments.size(); // Update number of segments
 }
 
 double EdgeDrawingImpl::NFA(double prob, int len)