# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1641972446 -3600
# Node ID c9ccd13c6a3c0bb23d748dc2adfccee5cc2f6280
# Parent  a2955abe4c2eb67b16f5198103f865d61db65b3e
cppcheck

diff -r a2955abe4c2e -r c9ccd13c6a3c Applications/Resources/RunCppCheck.sh
--- a/Applications/Resources/RunCppCheck.sh	Wed Jan 12 08:23:38 2022 +0100
+++ b/Applications/Resources/RunCppCheck.sh	Wed Jan 12 08:27:26 2022 +0100
@@ -30,8 +30,8 @@
 
 cat <<EOF > /tmp/cppcheck-suppressions.txt
 stlFindInsert:../../Applications/Samples/WebAssembly/SingleFrameViewer/SingleFrameViewerApplication.h
-stlFindInsert:../../Applications/StoneWebViewer/WebAssembly/StoneWebViewer.cpp:485
-stlFindInsert:../../Applications/StoneWebViewer/WebAssembly/StoneWebViewer.cpp:1123
+stlFindInsert:../../Applications/StoneWebViewer/WebAssembly/StoneWebViewer.cpp:508
+stlFindInsert:../../Applications/StoneWebViewer/WebAssembly/StoneWebViewer.cpp:1151
 unpreciseMathCall:../../OrthancStone/Sources/Scene2D/Internals/CairoFloatTextureRenderer.cpp
 unpreciseMathCall:../../OrthancStone/Sources/Scene2D/LookupTableTextureSceneLayer.cpp
 unreadVariable:../../OrthancStone/Sources/Viewport/SdlViewport.cpp:143
diff -r a2955abe4c2e -r c9ccd13c6a3c OrthancStone/Sources/Toolbox/UnionOfRectangles.cpp
--- a/OrthancStone/Sources/Toolbox/UnionOfRectangles.cpp	Wed Jan 12 08:23:38 2022 +0100
+++ b/OrthancStone/Sources/Toolbox/UnionOfRectangles.cpp	Wed Jan 12 08:27:26 2022 +0100
@@ -97,7 +97,7 @@
     Operation  operation_;
   
   public:
-    Visitor(Operation operation) :
+    explicit Visitor(Operation operation) :
       operation_(operation)
     {
     }
@@ -481,7 +481,7 @@
     horizontalJunctions.reserve(2 * verticalEdges.size());
 
     for (std::list<Internals::OrientedIntegerLine2D>::const_iterator
-           it = verticalEdges.begin(); it != verticalEdges.end(); it++)
+           it = verticalEdges.begin(); it != verticalEdges.end(); ++it)
     {
       assert(it->GetX1() == it->GetX2());
       horizontalJunctions.push_back(HorizontalJunction(it->GetX1(), it->GetY1(), it->GetY2(), it->IsDownward()));
@@ -518,13 +518,13 @@
     allEdges.reserve(horizontalEdges.size() + verticalEdges.size());
     
     for (std::list<Internals::OrientedIntegerLine2D>::const_iterator
-           it = horizontalEdges.begin(); it != horizontalEdges.end(); it++)
+           it = horizontalEdges.begin(); it != horizontalEdges.end(); ++it)
     {
       allEdges.push_back(*it);
     }
     
     for (std::list<Internals::OrientedIntegerLine2D>::const_iterator
-           it = verticalEdges.begin(); it != verticalEdges.end(); it++)
+           it = verticalEdges.begin(); it != verticalEdges.end(); ++it)
     {
       allEdges.push_back(*it);
     }