# HG changeset patch
# User am@osimis.io
# Date 1550056277 -3600
# Node ID baf8c8d68bbc32246645099ebcdfafdd09fb1472
# Parent  a95090305dd4b6bdd4ced1daf1f72992c64d8c8d
cleanup

diff -r a95090305dd4 -r baf8c8d68bbc Framework/Radiography/RadiographyMaskLayer.cpp
--- a/Framework/Radiography/RadiographyMaskLayer.cpp	Wed Feb 13 12:04:02 2019 +0100
+++ b/Framework/Radiography/RadiographyMaskLayer.cpp	Wed Feb 13 12:11:17 2019 +0100
@@ -252,16 +252,11 @@
     unsigned int top;
     unsigned int bottom;
 
-//    ComputeMaskExtent(left, right, top, bottom, corners_);
+    ComputeMaskExtent(left, right, top, bottom, corners_);
 
-    left = 0;
-    right = 2500;
-    top = 0;
-    bottom = 2500;
     // first fill the complete image
     Orthanc::ImageProcessing::Set(*mask_, OUT_MASK_VALUE);
 
-
     {
       // from http://alienryderflex.com/polygon_fill/
       std::auto_ptr<int> raiiNodeX(new int(corners_.size()));
@@ -279,8 +274,8 @@
 
         for (i = 0; i < (int)corners_.size(); i++)
         {
-          if ((int)corners_[i].y < pixelY && (int)corners_[j].y >=  pixelY
-          ||  (int)corners_[j].y < pixelY && (int)corners_[i].y >= pixelY)
+          if ((int)corners_[i].y <= pixelY && (int)corners_[j].y >=  pixelY
+          ||  (int)corners_[j].y <= pixelY && (int)corners_[i].y >= pixelY)
           {
             nodeX[nodes++]= (int)((double)corners_[i].x + ((double)pixelY - (double)corners_[i].y)/((double)corners_[j].y - (double)corners_[i].y) *((double)corners_[j].x - (double)corners_[i].x));
           }
@@ -289,11 +284,15 @@
 
         //  Sort the nodes, via a simple “Bubble” sort.
         i=0;
-        while (i<nodes-1)
+        while (i < nodes-1)
         {
-          if (nodeX[i]>nodeX[i+1])
+          if (nodeX[i] > nodeX[i+1])
           {
-            swap=nodeX[i]; nodeX[i]=nodeX[i+1]; nodeX[i+1]=swap; if (i) i--;
+            swap = nodeX[i];
+            nodeX[i] = nodeX[i+1];
+            nodeX[i+1] = swap;
+            if (i)
+              i--;
           }
           else
           {
@@ -307,13 +306,13 @@
         {
           if   (nodeX[i  ]>=(int)right)
             break;
-          if   (nodeX[i+1]> (int)left)
+          if   (nodeX[i+1]>= (int)left)
           {
             if (nodeX[i  ]< (int)left )
               nodeX[i  ]=(int)left ;
             if (nodeX[i+1]> (int)right)
               nodeX[i+1]=(int)right;
-            for (pixelX=nodeX[i]; pixelX<nodeX[i+1]; pixelX++)
+            for (pixelX = nodeX[i]; pixelX <= nodeX[i+1]; pixelX++)
             {
               *(row + pixelX) = IN_MASK_VALUE;
             }