Cleanup: misc spelling fixes
[blender.git] / source / blender / freestyle / intern / view_map / ViewMapBuilder.cpp
index 44d840c2cc5fca8a2ef5d800195f0ad6c26c3dcb..c67be4f97b359fa4e8ad50f8e0dc786509c20245 100644 (file)
@@ -93,22 +93,26 @@ static void findOccludee(FEdge *fe,
       if (0 != face) {
         skipFace = false;
 
-        if (face == oface)
+        if (face == oface) {
           continue;
+        }
 
-        if (faceVertices.empty())
+        if (faceVertices.empty()) {
           continue;
+        }
 
         for (vector<WVertex *>::iterator fv = faceVertices.begin(), fvend = faceVertices.end();
              fv != fvend;
              ++fv) {
-          if ((*fv)->isBoundary())
+          if ((*fv)->isBoundary()) {
             continue;
+          }
           WVertex::incoming_edge_iterator iebegin = (*fv)->incoming_edges_begin();
           WVertex::incoming_edge_iterator ieend = (*fv)->incoming_edges_end();
           for (ie = iebegin; ie != ieend; ++ie) {
-            if ((*ie) == 0)
+            if ((*ie) == 0) {
               continue;
+            }
 
             WFace *sface = (*ie)->GetbFace();
             if (sface == oface) {
@@ -116,16 +120,18 @@ static void findOccludee(FEdge *fe,
               break;
             }
           }
-          if (skipFace)
+          if (skipFace) {
             break;
+          }
         }
-        if (skipFace)
+        if (skipFace) {
           continue;
+        }
       }
       else {
         // check whether the edge and the polygon plane are coincident:
         //-------------------------------------------------------------
-        //first let us compute the plane equation.
+        // first let us compute the plane equation.
         if (GeomUtils::COINCIDENT ==
             GeomUtils::intersectRayPlane(origin, edgeDir, p->getNormal(), d, t, epsilon)) {
 #if LOGGING
@@ -146,7 +152,7 @@ static void findOccludee(FEdge *fe,
         }
 #endif
         if (fabs(v * p->getNormal()) > 0.0001) {
-          if ((t > 0.0)) {  // && (t<1.0))
+          if ((t > 0.0) /* && (t<1.0) */) {
             if (t < mint) {
               *oaWFace = oface;
               mint = t;
@@ -165,8 +171,9 @@ static void findOccludee(FEdge *fe,
       }
     }
 
-    if (noIntersection)
+    if (noIntersection) {
       *oaWFace = NULL;
+    }
   }
 }
 
@@ -249,14 +256,16 @@ static int computeVisibility(ViewMap *viewMap,
   WFace *oface;
   bool skipFace;
 
-  if (face)
+  if (face) {
     face->RetrieveVertexList(faceVertices);
+  }
 
   I occluders(grid, center, epsilon);
 
   for (occluders.initBeforeTarget(); occluders.validBeforeTarget(); occluders.nextOccluder()) {
-    // If we're dealing with an exact silhouette, check whether we must take care of this occluder of not.
-    // (Indeed, we don't consider the occluders that share at least one vertex with the face containing this edge).
+    // If we're dealing with an exact silhouette, check whether we must take care of this occluder
+    // of not. (Indeed, we don't consider the occluders that share at least one vertex with the
+    // face containing this edge).
     //-----------
     oface = occluders.getWFace();
     Polygon3r *p = occluders.getCameraSpacePolygon();
@@ -314,25 +323,28 @@ static int computeVisibility(ViewMap *viewMap,
       for (vector<WVertex *>::iterator fv = faceVertices.begin(), fvend = faceVertices.end();
            fv != fvend;
            ++fv) {
-        if ((*fv)->isBoundary())
+        if ((*fv)->isBoundary()) {
           continue;
+        }
 
         WVertex::incoming_edge_iterator iebegin = (*fv)->incoming_edges_begin();
         WVertex::incoming_edge_iterator ieend = (*fv)->incoming_edges_end();
         for (ie = iebegin; ie != ieend; ++ie) {
-          if ((*ie) == 0)
+          if ((*ie) == 0) {
             continue;
+          }
 
           WFace *sface = (*ie)->GetbFace();
-          //WFace *sfacea = (*ie)->GetaFace();
-          //if ((sface == oface) || (sfacea == oface))
+          // WFace *sfacea = (*ie)->GetaFace();
+          // if ((sface == oface) || (sfacea == oface))
           if (sface == oface) {
             skipFace = true;
             break;
           }
         }
-        if (skipFace)
+        if (skipFace) {
           break;
+        }
       }
       if (skipFace) {
 #if LOGGING
@@ -346,7 +358,7 @@ static int computeVisibility(ViewMap *viewMap,
     else {
       // check whether the edge and the polygon plane are coincident:
       //-------------------------------------------------------------
-      //first let us compute the plane equation.
+      // first let us compute the plane equation.
       if (GeomUtils::COINCIDENT ==
           GeomUtils::intersectRayPlane(origin, edgeDir, p->getNormal(), d, t, epsilon)) {
 #if LOGGING
@@ -394,8 +406,9 @@ static int computeVisibility(ViewMap *viewMap,
           }
           ++qi;
 
-          if (!grid.enableQI())
+          if (!grid.enableQI()) {
             break;
+          }
         }
 
         occluders.reportDepth(center, u, t);
@@ -412,11 +425,12 @@ static int computeVisibility(ViewMap *viewMap,
 
 // computeCumulativeVisibility returns the lowest x such that the majority of FEdges have QI <= x
 //
-// This was probably the original intention of the "normal" algorithm on which computeDetailedVisibility is based.
-// But because the "normal" algorithm chooses the most popular QI, without considering any other values, a ViewEdge
-// with FEdges having QIs of 0, 21, 22, 23, 24 and 25 will end up having a total QI of 0, even though most of the
-// FEdges are heavily occluded. computeCumulativeVisibility will treat this case as a QI of 22 because 3 out of
-// 6 occluders have QI <= 22.
+// This was probably the original intention of the "normal" algorithm on which
+// computeDetailedVisibility is based. But because the "normal" algorithm chooses the most popular
+// QI, without considering any other values, a ViewEdge with FEdges having QIs of 0, 21, 22, 23, 24
+// and 25 will end up having a total QI of 0, even though most of the FEdges are heavily occluded.
+// computeCumulativeVisibility will treat this case as a QI of 22 because 3 out of 6 occluders have
+// QI <= 22.
 
 template<typename G, typename I>
 static void computeCumulativeVisibility(ViewMap *ioViewMap,
@@ -438,8 +452,9 @@ static void computeCumulativeVisibility(ViewMap *ioViewMap,
   unsigned qiMajority;
   for (vector<ViewEdge *>::iterator ve = vedges.begin(), veend = vedges.end(); ve != veend; ve++) {
     if (iRenderMonitor) {
-      if (iRenderMonitor->testBreak())
+      if (iRenderMonitor->testBreak()) {
         break;
+      }
       if (cnt % cntStep == 0) {
         stringstream ss;
         ss << "Freestyle: Visibility computations " << (100 * cnt / vedges.size()) << "%";
@@ -513,7 +528,7 @@ static void computeCumulativeVisibility(ViewMap *ioViewMap,
         continue;
       }
       if ((maxCard < qiMajority)) {
-        //ARB: change &wFace to wFace and use reference in called function
+        // ARB: change &wFace to wFace and use reference in called function
         tmpQI = computeVisibility<G, I>(
             ioViewMap, fe, grid, epsilon, *ve, &wFace, &foundOccluders);
 #if LOGGING
@@ -522,11 +537,11 @@ static void computeCumulativeVisibility(ViewMap *ioViewMap,
         }
 #endif
 
-        //ARB: This is an error condition, not an alert condition.
+        // ARB: This is an error condition, not an alert condition.
         // Some sort of recovery or abort is necessary.
         if (tmpQI >= 256) {
           cerr << "Warning: too many occluding levels" << endl;
-          //ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
+          // ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
           tmpQI = 255;
         }
 
@@ -536,8 +551,8 @@ static void computeCumulativeVisibility(ViewMap *ioViewMap,
         }
       }
       else {
-        //ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
-        //ARB: change &wFace to wFace and use reference in called function
+        // ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
+        // ARB: change &wFace to wFace and use reference in called function
         findOccludee<G, I>(fe, grid, epsilon, *ve, &wFace);
 #if LOGGING
         if (_global.debug & G_DEBUG_FREESTYLE) {
@@ -589,8 +604,8 @@ static void computeCumulativeVisibility(ViewMap *ioViewMap,
       }
     }
     // occluders --
-    // I would rather not have to go through the effort of creating this set and then copying out its contents.
-    // Is there a reason why ViewEdge::_Occluders cannot be converted to a set<>?
+    // I would rather not have to go through the effort of creating this set and then copying out
+    // its contents. Is there a reason why ViewEdge::_Occluders cannot be converted to a set<>?
     for (set<ViewShape *>::iterator o = foundOccluders.begin(), oend = foundOccluders.end();
          o != oend;
          ++o) {
@@ -643,8 +658,9 @@ static void computeDetailedVisibility(ViewMap *ioViewMap,
   unsigned maxIndex, maxCard;
   unsigned qiMajority;
   for (vector<ViewEdge *>::iterator ve = vedges.begin(), veend = vedges.end(); ve != veend; ve++) {
-    if (iRenderMonitor && iRenderMonitor->testBreak())
+    if (iRenderMonitor && iRenderMonitor->testBreak()) {
       break;
+    }
 #if LOGGING
     if (_global.debug & G_DEBUG_FREESTYLE) {
       cout << "Processing ViewEdge " << (*ve)->getId() << endl;
@@ -710,7 +726,7 @@ static void computeDetailedVisibility(ViewMap *ioViewMap,
         continue;
       }
       if ((maxCard < qiMajority)) {
-        //ARB: change &wFace to wFace and use reference in called function
+        // ARB: change &wFace to wFace and use reference in called function
         tmpQI = computeVisibility<G, I>(
             ioViewMap, fe, grid, epsilon, *ve, &wFace, &foundOccluders);
 #if LOGGING
@@ -719,11 +735,11 @@ static void computeDetailedVisibility(ViewMap *ioViewMap,
         }
 #endif
 
-        //ARB: This is an error condition, not an alert condition.
+        // ARB: This is an error condition, not an alert condition.
         // Some sort of recovery or abort is necessary.
         if (tmpQI >= 256) {
           cerr << "Warning: too many occluding levels" << endl;
-          //ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
+          // ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
           tmpQI = 255;
         }
 
@@ -733,8 +749,8 @@ static void computeDetailedVisibility(ViewMap *ioViewMap,
         }
       }
       else {
-        //ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
-        //ARB: change &wFace to wFace and use reference in called function
+        // ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
+        // ARB: change &wFace to wFace and use reference in called function
         findOccludee<G, I>(fe, grid, epsilon, *ve, &wFace);
 #if LOGGING
         if (_global.debug & G_DEBUG_FREESTYLE) {
@@ -779,8 +795,8 @@ static void computeDetailedVisibility(ViewMap *ioViewMap,
     // qi --
     (*ve)->setQI(maxIndex);
     // occluders --
-    // I would rather not have to go through the effort of creating this this set and then copying out its contents.
-    // Is there a reason why ViewEdge::_Occluders cannot be converted to a set<>?
+    // I would rather not have to go through the effort of creating this this set and then copying
+    // out its contents. Is there a reason why ViewEdge::_Occluders cannot be converted to a set<>?
     for (set<ViewShape *>::iterator o = foundOccluders.begin(), oend = foundOccluders.end();
          o != oend;
          ++o) {
@@ -877,15 +893,15 @@ static void computeFastVisibility(ViewMap *ioViewMap, G &grid, real epsilon)
       }
       if (even_test) {
         if ((maxCard < qiMajority)) {
-          //ARB: change &wFace to wFace and use reference in called function
+          // ARB: change &wFace to wFace and use reference in called function
           tmpQI = computeVisibility<G, I>(
               ioViewMap, fe, grid, epsilon, *ve, &wFace, &foundOccluders);
 
-          //ARB: This is an error condition, not an alert condition.
+          // ARB: This is an error condition, not an alert condition.
           // Some sort of recovery or abort is necessary.
           if (tmpQI >= 256) {
             cerr << "Warning: too many occluding levels" << endl;
-            //ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
+            // ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
             tmpQI = 255;
           }
 
@@ -895,8 +911,8 @@ static void computeFastVisibility(ViewMap *ioViewMap, G &grid, real epsilon)
           }
         }
         else {
-          //ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
-          //ARB: change &wFace to wFace and use reference in called function
+          // ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
+          // ARB: change &wFace to wFace and use reference in called function
           findOccludee<G, I>(fe, grid, epsilon, *ve, &wFace);
         }
 
@@ -1008,7 +1024,8 @@ void ViewMapBuilder::BuildGrid(WingedEdge &we, const BBox<Vec3r> &bbox, unsigned
   Vec3r size;
   for (unsigned int i = 0; i < 3; i++) {
     size[i] = fabs(bbox.getMax()[i] - bbox.getMin()[i]);
-    // let make the grid 1/10 bigger to avoid numerical errors while computing triangles/cells intersections.
+    // let make the grid 1/10 bigger to avoid numerical errors while computing triangles/cells
+    // intersections.
     size[i] += size[i] / 10.0;
     if (size[i] == 0) {
       if (_global.debug & G_DEBUG_FREESTYLE) {
@@ -1083,8 +1100,8 @@ void ViewMapBuilder::CullViewEdges(ViewMap *ioViewMap,
 
   // Non-displayable view edges will be skipped over during visibility calculation.
 
-  // View edges will be culled according to their position w.r.t. the viewport proscenium (viewport + 5% border,
-  // or some such).
+  // View edges will be culled according to their position w.r.t. the viewport proscenium (viewport
+  // + 5% border, or some such).
 
   // Get proscenium boundary for culling
   GridHelpers::getDefaultViewProscenium(viewProscenium);
@@ -1098,17 +1115,16 @@ void ViewMapBuilder::CullViewEdges(ViewMap *ioViewMap,
     cout << "Origin: [" << prosceniumOrigin[0] << ", " << prosceniumOrigin[1] << "]" << endl;
   }
 
-  // A separate occluder proscenium will also be maintained, starting out the same as the viewport proscenium, and
-  // expanding as necessary so that it encompasses the center point of at least one feature edge in each retained view
-  // edge.
-  // The occluder proscenium will be used later to cull occluding triangles before they are inserted into the Grid.
-  // The occluder proscenium starts out the same size as the view proscenium
+  // A separate occluder proscenium will also be maintained, starting out the same as the viewport
+  // proscenium, and expanding as necessary so that it encompasses the center point of at least one
+  // feature edge in each retained view edge. The occluder proscenium will be used later to cull
+  // occluding triangles before they are inserted into the Grid. The occluder proscenium starts out
+  // the same size as the view proscenium
   GridHelpers::getDefaultViewProscenium(occluderProscenium);
 
-  // N.B. Freestyle is inconsistent in its use of ViewMap::viewedges_container and vector<ViewEdge*>::iterator.
-  // Probably all occurences of vector<ViewEdge*>::iterator should be replaced ViewMap::viewedges_container
-  // throughout the code.
-  // For each view edge
+  // N.B. Freestyle is inconsistent in its use of ViewMap::viewedges_container and
+  // vector<ViewEdge*>::iterator. Probably all occurrences of vector<ViewEdge*>::iterator should be
+  // replaced ViewMap::viewedges_container throughout the code. For each view edge
   ViewMap::viewedges_container::iterator ve, veend;
 
   for (ve = ioViewMap->ViewEdges().begin(), veend = ioViewMap->ViewEdges().end(); ve != veend;
@@ -1121,7 +1137,7 @@ void ViewMapBuilder::CullViewEdges(ViewMap *ioViewMap,
     //        If none exists, find the feature edge with center point closest to viewport origin.
     //            Expand occluder proscenium to enclose center point.
 
-    // For each feature edge, while bestOccluderTarget not found and view edge not visibile
+    // For each feature edge, while bestOccluderTarget not found and view edge not visible
     bool bestOccluderTargetFound = false;
     FEdge *bestOccluderTarget = NULL;
     real bestOccluderDistance = 0.0;
@@ -1130,8 +1146,8 @@ void ViewMapBuilder::CullViewEdges(ViewMap *ioViewMap,
     // All ViewEdges start culled
     (*ve)->setIsInImage(false);
 
-    // For simple visibility calculation: mark a feature edge that is known to have a center point inside the
-    // occluder proscenium. Cull all other feature edges.
+    // For simple visibility calculation: mark a feature edge that is known to have a center point
+    // inside the occluder proscenium. Cull all other feature edges.
     do {
       // All FEdges start culled
       fe->setIsInImage(false);
@@ -1165,15 +1181,15 @@ void ViewMapBuilder::CullViewEdges(ViewMap *ioViewMap,
       fe = fe->nextEdge();
     } while (fe && fe != festart && !(bestOccluderTargetFound && (*ve)->isInImage()));
 
-    // Either we have run out of FEdges, or we already have the one edge we need to determine visibility
-    // Cull all remaining edges.
+    // Either we have run out of FEdges, or we already have the one edge we need to determine
+    // visibility Cull all remaining edges.
     while (fe && fe != festart) {
       fe->setIsInImage(false);
       fe = fe->nextEdge();
     }
 
-    // If bestOccluderTarget was not found inside the occluder proscenium, we need to expand the occluder
-    // proscenium to include it.
+    // If bestOccluderTarget was not found inside the occluder proscenium, we need to expand the
+    // occluder proscenium to include it.
     if ((*ve)->isInImage() && bestOccluderTarget != NULL && !bestOccluderTargetFound) {
       // Expand occluder proscenium to enclose bestOccluderTarget
       Vec3r point = bestOccluderTarget->center2d();
@@ -1204,14 +1220,15 @@ void ViewMapBuilder::CullViewEdges(ViewMap *ioViewMap,
 
   // For "Normal" or "Fast" style visibility computation only:
 
-  // For more detailed visibility calculation, make a second pass through the view map, marking all feature edges
-  // with center points inside the final occluder proscenium. All of these feature edges can be considered during
-  // visibility calculation.
+  // For more detailed visibility calculation, make a second pass through the view map, marking all
+  // feature edges with center points inside the final occluder proscenium. All of these feature
+  // edges can be considered during visibility calculation.
 
-  // So far we have only found one FEdge per ViewEdge.  The "Normal" and "Fast" styles of visibility computation
-  // want to consider many FEdges for each ViewEdge.
-  // Here we re-scan the view map to find any usable FEdges that we skipped on the first pass, or that have become
-  // usable because the occluder proscenium has been expanded since the edge was visited on the first pass.
+  // So far we have only found one FEdge per ViewEdge.  The "Normal" and "Fast" styles of
+  // visibility computation want to consider many FEdges for each ViewEdge. Here we re-scan the
+  // view map to find any usable FEdges that we skipped on the first pass, or that have become
+  // usable because the occluder proscenium has been expanded since the edge was visited on the
+  // first pass.
   if (extensiveFEdgeSearch) {
     // For each view edge,
     for (ve = ioViewMap->ViewEdges().begin(), veend = ioViewMap->ViewEdges().end(); ve != veend;
@@ -1240,8 +1257,9 @@ void ViewMapBuilder::computeInitialViewEdges(WingedEdge &we)
   SShape *psShape;
 
   for (vector<WShape *>::const_iterator it = wshapes.begin(); it != wshapes.end(); it++) {
-    if (_pRenderMonitor && _pRenderMonitor->testBreak())
+    if (_pRenderMonitor && _pRenderMonitor->testBreak()) {
       break;
+    }
 
     // create the embedding
     psShape = new SShape;
@@ -1283,10 +1301,12 @@ void ViewMapBuilder::computeCusps(ViewMap *ioViewMap)
   ViewMap::viewedges_container &vedges = ioViewMap->ViewEdges();
   ViewMap::viewedges_container::iterator ve = vedges.begin(), veend = vedges.end();
   for (; ve != veend; ++ve) {
-    if (_pRenderMonitor && _pRenderMonitor->testBreak())
+    if (_pRenderMonitor && _pRenderMonitor->testBreak()) {
       break;
-    if ((!((*ve)->getNature() & Nature::SILHOUETTE)) || (!((*ve)->fedgeA()->isSmooth())))
+    }
+    if ((!((*ve)->getNature() & Nature::SILHOUETTE)) || (!((*ve)->fedgeA()->isSmooth()))) {
       continue;
+    }
     FEdge *fe = (*ve)->fedgeA();
     FEdge *fefirst = fe;
     bool first = true;
@@ -1309,10 +1329,12 @@ void ViewMapBuilder::computeCusps(ViewMap *ioViewMap)
       }
       viewvector.normalize();
       if (first) {
-        if (((crossP) * (viewvector)) > 0)
+        if (((crossP) * (viewvector)) > 0) {
           positive = true;
-        else
+        }
+        else {
           positive = false;
+        }
         first = false;
       }
       // If we're in a positive part, we need a stronger negative value to change
@@ -1324,8 +1346,9 @@ void ViewMapBuilder::computeCusps(ViewMap *ioViewMap)
           // creates and insert cusp
           cusp = dynamic_cast<NonTVertex *>(
               ioViewMap->InsertViewVertex(fes->vertexA(), newVEdges));
-          if (cusp)
+          if (cusp) {
             cusp->setNature(cusp->getNature() | Nature::CUSP);
+          }
         }
       }
       else {
@@ -1334,8 +1357,9 @@ void ViewMapBuilder::computeCusps(ViewMap *ioViewMap)
           positive = true;
           cusp = dynamic_cast<NonTVertex *>(
               ioViewMap->InsertViewVertex(fes->vertexA(), newVEdges));
-          if (cusp)
+          if (cusp) {
             cusp->setNature(cusp->getNature() | Nature::CUSP);
+          }
         }
       }
       fe = fe->nextEdge();
@@ -1465,15 +1489,15 @@ void ViewMapBuilder::ComputeEdgesVisibility(ViewMap *ioViewMap,
         ComputeDetailedVisibility(ioViewMap, we, bbox, epsilon, true, factory);
       }
       catch (...) {
-        // Last resort catch to make sure RAII semantics hold for OptimizedGrid. Can be replaced with
-        // try...catch block around main() if the program as a whole is converted to RAII
+        // Last resort catch to make sure RAII semantics hold for OptimizedGrid. Can be replaced
+        // with try...catch block around main() if the program as a whole is converted to RAII
 
-        // This is the little-mentioned caveat of RAII: RAII does not work unless destructors are always
-        // called, but destructors are only called if all exceptions are caught (or std::terminate() is
-        // replaced).
+        // This is the little-mentioned caveat of RAII: RAII does not work unless destructors are
+        // always called, but destructors are only called if all exceptions are caught (or
+        // std::terminate() is replaced).
 
-        // We don't actually handle the exception here, so re-throw it now that our destructors have had a
-        // chance to run.
+        // We don't actually handle the exception here, so re-throw it now that our destructors
+        // have had a chance to run.
         throw;
       }
       break;
@@ -1554,8 +1578,9 @@ void ViewMapBuilder::ComputeRayCastingVisibility(ViewMap *ioViewMap, real epsilo
   unsigned qiMajority;
   static unsigned timestamp = 1;
   for (vector<ViewEdge *>::iterator ve = vedges.begin(), veend = vedges.end(); ve != veend; ve++) {
-    if (_pRenderMonitor && _pRenderMonitor->testBreak())
+    if (_pRenderMonitor && _pRenderMonitor->testBreak()) {
       break;
+    }
 #if LOGGING
     if (_global.debug & G_DEBUG_FREESTYLE) {
       cout << "Processing ViewEdge " << (*ve)->getId() << endl;
@@ -1591,11 +1616,11 @@ void ViewMapBuilder::ComputeRayCastingVisibility(ViewMap *ioViewMap, real epsilo
           cout << "\tFEdge: visibility " << tmpQI << endl;
         }
 #endif
-        //ARB: This is an error condition, not an alert condition.
+        // ARB: This is an error condition, not an alert condition.
         // Some sort of recovery or abort is necessary.
         if (tmpQI >= 256) {
           cerr << "Warning: too many occluding levels" << endl;
-          //ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
+          // ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
           tmpQI = 255;
         }
 
@@ -1605,7 +1630,7 @@ void ViewMapBuilder::ComputeRayCastingVisibility(ViewMap *ioViewMap, real epsilo
         }
       }
       else {
-        //ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
+        // ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
         FindOccludee(fe, _Grid, epsilon, &aFace, timestamp++);
 #if LOGGING
         if (_global.debug & G_DEBUG_FREESTYLE) {
@@ -1626,7 +1651,8 @@ void ViewMapBuilder::ComputeRayCastingVisibility(ViewMap *ioViewMap, real epsilo
 #endif
       }
       else {
-        //ARB: We are arbitrarily using the last observed value for occludee (almost always the value observed
+        // ARB: We are arbitrarily using the last observed value for occludee (almost always the
+        // value observed
         //     for the edge before festart). Is that meaningful?
         // ...in fact, _occludeeEmpty seems to be unused.
         fe->setOccludeeEmpty(true);
@@ -1645,8 +1671,10 @@ void ViewMapBuilder::ComputeRayCastingVisibility(ViewMap *ioViewMap, real epsilo
     // qi --
     (*ve)->setQI(maxIndex);
     // occluders --
-    for (set<ViewShape *>::iterator o = occluders.begin(), oend = occluders.end(); o != oend; ++o)
+    for (set<ViewShape *>::iterator o = occluders.begin(), oend = occluders.end(); o != oend;
+         ++o) {
       (*ve)->AddOccluder((*o));
+    }
 #if LOGGING
     if (_global.debug & G_DEBUG_FREESTYLE) {
       cout << "\tConclusion: QI = " << maxIndex << ", " << (*ve)->occluders_size() << " occluders."
@@ -1708,8 +1736,9 @@ void ViewMapBuilder::ComputeFastRayCastingVisibility(ViewMap *ioViewMap, real ep
   static unsigned timestamp = 1;
   bool even_test;
   for (vector<ViewEdge *>::iterator ve = vedges.begin(), veend = vedges.end(); ve != veend; ve++) {
-    if (_pRenderMonitor && _pRenderMonitor->testBreak())
+    if (_pRenderMonitor && _pRenderMonitor->testBreak()) {
       break;
+    }
 
     festart = (*ve)->fedgeA();
     fe = (*ve)->fedgeA();
@@ -1718,10 +1747,12 @@ void ViewMapBuilder::ComputeFastRayCastingVisibility(ViewMap *ioViewMap, real ep
       qiMajority++;
       fe = fe->nextEdge();
     } while (fe && fe != festart);
-    if (qiMajority >= 4)
+    if (qiMajority >= 4) {
       qiMajority >>= 2;
-    else
+    }
+    else {
       qiMajority = 1;
+    }
 
     set<ViewShape *> occluders;
 
@@ -1736,11 +1767,11 @@ void ViewMapBuilder::ComputeFastRayCastingVisibility(ViewMap *ioViewMap, real ep
         if ((maxCard < qiMajority)) {
           tmpQI = ComputeRayCastingVisibility(fe, _Grid, epsilon, occluders, &aFace, timestamp++);
 
-          //ARB: This is an error condition, not an alert condition.
+          // ARB: This is an error condition, not an alert condition.
           // Some sort of recovery or abort is necessary.
           if (tmpQI >= 256) {
             cerr << "Warning: too many occluding levels" << endl;
-            //ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
+            // ARB: Wild guess: instead of aborting or corrupting memory, treat as tmpQI == 255
             tmpQI = 255;
           }
 
@@ -1750,7 +1781,7 @@ void ViewMapBuilder::ComputeFastRayCastingVisibility(ViewMap *ioViewMap, real ep
           }
         }
         else {
-          //ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
+          // ARB: FindOccludee is redundant if ComputeRayCastingVisibility has been called
           FindOccludee(fe, _Grid, epsilon, &aFace, timestamp++);
         }
 
@@ -1789,7 +1820,9 @@ void ViewMapBuilder::ComputeFastRayCastingVisibility(ViewMap *ioViewMap, real ep
         }
         if (sameShape)
 #endif
-        (*ve)->setaShape(vshape);
+        {
+          (*ve)->setaShape(vshape);
+        }
       }
     }
 
@@ -1830,8 +1863,9 @@ void ViewMapBuilder::ComputeVeryFastRayCastingVisibility(ViewMap *ioViewMap, rea
   Polygon3r *aFace = NULL;
   static unsigned timestamp = 1;
   for (vector<ViewEdge *>::iterator ve = vedges.begin(), veend = vedges.end(); ve != veend; ve++) {
-    if (_pRenderMonitor && _pRenderMonitor->testBreak())
+    if (_pRenderMonitor && _pRenderMonitor->testBreak()) {
       break;
+    }
 
     set<ViewShape *> occluders;
 
@@ -1895,7 +1929,7 @@ void ViewMapBuilder::FindOccludee(FEdge *fe,
     for (p = occluders.begin(), pend = occluders.end(); p != pend; p++) {
       // check whether the edge and the polygon plane are coincident:
       //-------------------------------------------------------------
-      //first let us compute the plane equation.
+      // first let us compute the plane equation.
       oface = (WFace *)(*p)->userdata;
       Vec3r v1(((*p)->getVertices())[0]);
       Vec3r normal((*p)->getNormal());
@@ -1905,22 +1939,26 @@ void ViewMapBuilder::FindOccludee(FEdge *fe,
       if (face) {
         skipFace = false;
 
-        if (face == oface)
+        if (face == oface) {
           continue;
+        }
 
-        if (faceVertices.empty())
+        if (faceVertices.empty()) {
           continue;
+        }
 
         for (vector<WVertex *>::iterator fv = faceVertices.begin(), fvend = faceVertices.end();
              fv != fvend;
              ++fv) {
-          if ((*fv)->isBoundary())
+          if ((*fv)->isBoundary()) {
             continue;
+          }
           WVertex::incoming_edge_iterator iebegin = (*fv)->incoming_edges_begin();
           WVertex::incoming_edge_iterator ieend = (*fv)->incoming_edges_end();
           for (ie = iebegin; ie != ieend; ++ie) {
-            if ((*ie) == 0)
+            if ((*ie) == 0) {
               continue;
+            }
 
             WFace *sface = (*ie)->GetbFace();
             if (sface == oface) {
@@ -1928,16 +1966,19 @@ void ViewMapBuilder::FindOccludee(FEdge *fe,
               break;
             }
           }
-          if (skipFace)
+          if (skipFace) {
             break;
+          }
         }
-        if (skipFace)
+        if (skipFace) {
           continue;
+        }
       }
       else {
         if (GeomUtils::COINCIDENT ==
-            GeomUtils::intersectRayPlane(origin, edgeDir, normal, d, t, epsilon))
+            GeomUtils::intersectRayPlane(origin, edgeDir, normal, d, t, epsilon)) {
           continue;
+        }
       }
       if ((*p)->rayIntersect(A, v, t, t_u, t_v)) {
         if (fabs(v * normal) > 0.0001) {
@@ -1953,8 +1994,9 @@ void ViewMapBuilder::FindOccludee(FEdge *fe,
       }
     }
 
-    if (noIntersection)
+    if (noIntersection) {
       *oaPolygon = NULL;
+    }
   }
 }
 
@@ -1978,9 +2020,10 @@ void ViewMapBuilder::FindOccludee(
     u = Vec3r(_viewpoint - A);
   }
   u.normalize();
-  if (A < iGrid->getOrigin())
+  if (A < iGrid->getOrigin()) {
     cerr << "Warning: point is out of the grid for fedge " << fe->getId().getFirst() << "-"
          << fe->getId().getSecond() << endl;
+  }
 
   vector<WVertex *> faceVertices;
 
@@ -1989,8 +2032,9 @@ void ViewMapBuilder::FindOccludee(
     FEdgeSmooth *fes = dynamic_cast<FEdgeSmooth *>(fe);
     face = (WFace *)fes->face();
   }
-  if (face)
+  if (face) {
     face->RetrieveVertexList(faceVertices);
+  }
 
   return FindOccludee(
       fe, iGrid, epsilon, oaPolygon, timestamp, u, A, origin, edgeDir, faceVertices);
@@ -2022,7 +2066,7 @@ int ViewMapBuilder::ComputeRayCastingVisibility(FEdge *fe,
       (center.z() < gridOrigin.z()) || (center.x() > gridExtremity.x()) ||
       (center.y() > gridExtremity.y()) || (center.z() > gridExtremity.z())) {
     cerr << "Warning: point is out of the grid for fedge " << fe->getId() << endl;
-    //return 0;
+    // return 0;
   }
 
 #if 0
@@ -2069,12 +2113,14 @@ int ViewMapBuilder::ComputeRayCastingVisibility(FEdge *fe,
   WFace *oface;
   bool skipFace;
   OccludersSet::iterator p, pend;
-  if (face)
+  if (face) {
     face->RetrieveVertexList(faceVertices);
+  }
 
   for (p = occluders.begin(), pend = occluders.end(); p != pend; p++) {
-    // If we're dealing with an exact silhouette, check whether we must take care of this occluder of not.
-    // (Indeed, we don't consider the occluders that share at least one vertex with the face containing this edge).
+    // If we're dealing with an exact silhouette, check whether we must take care of this occluder
+    // of not. (Indeed, we don't consider the occluders that share at least one vertex with the
+    // face containing this edge).
     //-----------
     oface = (WFace *)(*p)->userdata;
 #if LOGGING
@@ -2116,25 +2162,28 @@ int ViewMapBuilder::ComputeRayCastingVisibility(FEdge *fe,
       for (vector<WVertex *>::iterator fv = faceVertices.begin(), fvend = faceVertices.end();
            fv != fvend;
            ++fv) {
-        if ((*fv)->isBoundary())
+        if ((*fv)->isBoundary()) {
           continue;
+        }
 
         WVertex::incoming_edge_iterator iebegin = (*fv)->incoming_edges_begin();
         WVertex::incoming_edge_iterator ieend = (*fv)->incoming_edges_end();
         for (ie = iebegin; ie != ieend; ++ie) {
-          if ((*ie) == 0)
+          if ((*ie) == 0) {
             continue;
+          }
 
           WFace *sface = (*ie)->GetbFace();
-          //WFace *sfacea = (*ie)->GetaFace();
-          //if ((sface == oface) || (sfacea == oface)) {
+          // WFace *sfacea = (*ie)->GetaFace();
+          // if ((sface == oface) || (sfacea == oface)) {
           if (sface == oface) {
             skipFace = true;
             break;
           }
         }
-        if (skipFace)
+        if (skipFace) {
           break;
+        }
       }
       if (skipFace) {
 #if LOGGING
@@ -2148,7 +2197,7 @@ int ViewMapBuilder::ComputeRayCastingVisibility(FEdge *fe,
     else {
       // check whether the edge and the polygon plane are coincident:
       //-------------------------------------------------------------
-      //first let us compute the plane equation.
+      // first let us compute the plane equation.
 
       if (GeomUtils::COINCIDENT ==
           GeomUtils::intersectRayPlane(origin, edgeDir, normal, d, t, epsilon)) {
@@ -2180,8 +2229,9 @@ int ViewMapBuilder::ComputeRayCastingVisibility(FEdge *fe,
           ViewShape *vshape = _ViewMap->viewShape(f->GetVertex(0)->shape()->GetId());
           oOccluders.insert(vshape);
           ++qi;
-          if (!_EnableQI)
+          if (!_EnableQI) {
             break;
+          }
         }
       }
     }
@@ -2236,12 +2286,15 @@ struct less_SVertex2D : public binary_function<SVertex *, SVertex *, bool> {
     Vec3r A = x->point2D();
     Vec3r B = y->point2D();
     for (unsigned int i = 0; i < 3; i++) {
-      if ((fabs(A[i] - B[i])) < epsilon)
+      if ((fabs(A[i] - B[i])) < epsilon) {
         continue;
-      if (A[i] < B[i])
+      }
+      if (A[i] < B[i]) {
         return true;
-      if (A[i] > B[i])
+      }
+      if (A[i] > B[i]) {
         return false;
+      }
     }
     return false;
   }
@@ -2262,8 +2315,9 @@ struct less_Intersection : public binary_function<intersection *, intersection *
   {
     real tx = x->getParameter(edge);
     real ty = y->getParameter(edge);
-    if (tx > ty)
+    if (tx > ty) {
       return true;
+    }
     return false;
   }
 };
@@ -2336,8 +2390,9 @@ void ViewMapBuilder::ComputeSweepLineIntersections(ViewMap *ioViewMap, real epsi
   vector<segment *> vsegments;
   for (vector<SVertex *>::iterator sv = svertices.begin(), svend = svertices.end(); sv != svend;
        sv++) {
-    if (_pRenderMonitor && _pRenderMonitor->testBreak())
+    if (_pRenderMonitor && _pRenderMonitor->testBreak()) {
       break;
+    }
 
     const vector<FEdge *> &vedges = (*sv)->fedges();
 
@@ -2373,8 +2428,9 @@ void ViewMapBuilder::ComputeSweepLineIntersections(ViewMap *ioViewMap, real epsi
   }
 
   // reset userdata:
-  for (fe = ioEdges.begin(), fend = ioEdges.end(); fe != fend; fe++)
+  for (fe = ioEdges.begin(), fend = ioEdges.end(); fe != fend; fe++) {
     (*fe)->userdata = NULL;
+  }
 
   // list containing the new edges resulting from splitting operations.
   vector<FEdge *> newEdges;
@@ -2404,24 +2460,28 @@ void ViewMapBuilder::ComputeSweepLineIntersections(ViewMap *ioViewMap, real epsi
     real ta = (*i)->tA;
     real tb = (*i)->tB;
 
-    if ((ta < -epsilon) || (ta > 1 + epsilon))
+    if ((ta < -epsilon) || (ta > 1 + epsilon)) {
       cerr << "Warning: 2D intersection out of range for edge " << fA->vertexA()->getId() << " - "
            << fA->vertexB()->getId() << endl;
+    }
 
-    if ((tb < -epsilon) || (tb > 1 + epsilon))
+    if ((tb < -epsilon) || (tb > 1 + epsilon)) {
       cerr << "Warning: 2D intersection out of range for edge " << fB->vertexA()->getId() << " - "
            << fB->vertexB()->getId() << endl;
+    }
 
     real Ta = SilhouetteGeomEngine::ImageToWorldParameter(fA, ta);
     real Tb = SilhouetteGeomEngine::ImageToWorldParameter(fB, tb);
 
-    if ((Ta < -epsilon) || (Ta > 1 + epsilon))
+    if ((Ta < -epsilon) || (Ta > 1 + epsilon)) {
       cerr << "Warning: 3D intersection out of range for edge " << fA->vertexA()->getId() << " - "
            << fA->vertexB()->getId() << endl;
+    }
 
-    if ((Tb < -epsilon) || (Tb > 1 + epsilon))
+    if ((Tb < -epsilon) || (Tb > 1 + epsilon)) {
       cerr << "Warning: 3D intersection out of range for edge " << fB->vertexA()->getId() << " - "
            << fB->vertexB()->getId() << endl;
+    }
 
 #if 0
     if (_global.debug & G_DEBUG_FREESTYLE) {
@@ -2432,10 +2492,12 @@ void ViewMapBuilder::ComputeSweepLineIntersections(ViewMap *ioViewMap, real epsi
         printf("b1 %e, %e -- b2 %e, %e\n", b1[0], b1[1], b2[0], b2[1]);
         //printf("line([%e, %e], [%e, %e]);\n", a1[0], a2[0], a1[1], a2[1]);
         //printf("line([%e, %e], [%e, %e]);\n", b1[0], b2[0], b1[1], b2[1]);
-        if ((Ta < -epsilon) || (Ta > 1 + epsilon))
+        if ((Ta < -epsilon) || (Ta > 1 + epsilon)) {
           printf("Ta %.12e\n", Ta);
-        if ((Tb < -epsilon) || (Tb > 1 + epsilon))
+        }
+        if ((Tb < -epsilon) || (Tb > 1 + epsilon)) {
           printf("Tb %.12e\n", Tb);
+        }
         printf("A1 %e, %e, %e -- A2 %e, %e, %e\n", A1[0], A1[1], A1[2], A2[0], A2[1], A2[2]);
         printf("B1 %e, %e, %e -- B2 %e, %e, %e\n", B1[0], B1[1], B1[2], B2[0], B2[1], B2[2]);
       }
@@ -2483,8 +2545,9 @@ void ViewMapBuilder::ComputeSweepLineIntersections(ViewMap *ioViewMap, real epsi
     vector<intersection *> &eIntersections = (*s)->intersections();
     // we first need to sort these intersections from farther to closer to A
     sort(eIntersections.begin(), eIntersections.end(), less_Intersection(*s));
-    for (i = eIntersections.begin(), iend = eIntersections.end(); i != iend; i++)
+    for (i = eIntersections.begin(), iend = eIntersections.end(); i != iend; i++) {
       edgeVVertices.push_back((TVertex *)(*i)->userdata);
+    }
 
     shape->SplitEdge(fedge, edgeVVertices, ioViewMap->FEdges(), ioViewMap->ViewEdges());
 
@@ -2498,8 +2561,9 @@ void ViewMapBuilder::ComputeSweepLineIntersections(ViewMap *ioViewMap, real epsi
   }
 
   // reset userdata:
-  for (fe = ioEdges.begin(), fend = ioEdges.end(); fe != fend; fe++)
+  for (fe = ioEdges.begin(), fend = ioEdges.end(); fe != fend; fe++) {
     (*fe)->userdata = NULL;
+  }
 
   // delete segments
   if (!segments.empty()) {