diff OrthancStone/Sources/Loaders/OrthancMultiframeVolumeLoader.cpp @ 1571:85e117739eca

cppcheck
author Sebastien Jodogne <s.jodogne@gmail.com>
date Mon, 21 Sep 2020 17:46:39 +0200
parents 244ad1e4e76a
children 4fb8fdf03314
line wrap: on
line diff
--- a/OrthancStone/Sources/Loaders/OrthancMultiframeVolumeLoader.cpp	Mon Sep 21 14:50:26 2020 +0200
+++ b/OrthancStone/Sources/Loaders/OrthancMultiframeVolumeLoader.cpp	Mon Sep 21 17:46:39 2020 +0200
@@ -44,7 +44,7 @@
 
     }
 
-    virtual void Handle(const OrthancStone::OrthancRestApiCommand::SuccessMessage& message)
+    virtual void Handle(const OrthancStone::OrthancRestApiCommand::SuccessMessage& message) ORTHANC_OVERRIDE
     {
       // Complete the DICOM tags with just-received "Grid Frame Offset Vector"
       std::string s = Orthanc::Toolbox::StripSpaces(message.GetAnswer());
@@ -73,8 +73,8 @@
   class OrthancMultiframeVolumeLoader::LoadGeometry : public State
   {
   public:
-    LoadGeometry(OrthancMultiframeVolumeLoader& that) :
-    State(that)
+    explicit LoadGeometry(OrthancMultiframeVolumeLoader& that) :
+      State(that)
     {
     }
       
@@ -115,8 +115,8 @@
   class OrthancMultiframeVolumeLoader::LoadTransferSyntax : public State
   {
   public:
-    LoadTransferSyntax(OrthancMultiframeVolumeLoader& that) :
-    State(that)
+    explicit LoadTransferSyntax(OrthancMultiframeVolumeLoader& that) :
+      State(that)
     {
     }
       
@@ -129,8 +129,8 @@
   class OrthancMultiframeVolumeLoader::LoadUncompressedPixelData : public State
   {
   public:
-    LoadUncompressedPixelData(OrthancMultiframeVolumeLoader& that) :
-    State(that)
+    explicit LoadUncompressedPixelData(OrthancMultiframeVolumeLoader& that) :
+      State(that)
     {
     }
       
@@ -385,7 +385,7 @@
           T pixelValue = it->first;
           uint64_t count = it->second;
           totalCount += count;
-          it++;
+          ++it;
           if (it == distribution.end())
             distributionRawMax_ = static_cast<float>(pixelValue);
         }
@@ -443,7 +443,7 @@
             currentCount += count;
           }
           // and continue walking along the distribution
-          it++;
+          ++it;
         }
       }
 
@@ -473,7 +473,7 @@
             currentCount += count;
           }
           // and continue walking along the distribution
-          it++;
+          ++it;
         }
       }
       if (resultMin > resultMax)