# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1752248008 -7200
# Node ID 93d2e634fc6d6b2f86d265f9f2a996827717e64f
# Parent  f1fe166b21d5cde322a72ec5401a7daf8e6804e9
cppcheck

diff -r f1fe166b21d5 -r 93d2e634fc6d OrthancFramework/Sources/HttpServer/HttpServer.cpp
--- a/OrthancFramework/Sources/HttpServer/HttpServer.cpp	Fri Jul 11 17:27:37 2025 +0200
+++ b/OrthancFramework/Sources/HttpServer/HttpServer.cpp	Fri Jul 11 17:33:28 2025 +0200
@@ -1421,7 +1421,7 @@
     if (method == HttpMethod_Post ||
         method == HttpMethod_Put)
     {
-      PostDataStatus status;
+      PostDataStatus postStatus;
 
       bool isMultipartForm = false;
 
@@ -1438,8 +1438,8 @@
          **/
         isMultipartForm = true;
 
-        status = ReadBodyToString(body, connection, headers);
-        if (status == PostDataStatus_Success)
+        postStatus = ReadBodyToString(body, connection, headers);
+        if (postStatus == PostDataStatus_Success)
         {
           server.ProcessMultipartFormData(remoteIp, username, uri, headers, body, boundary, authenticationPayload);
           output.SendStatus(HttpStatus_200_Ok);
@@ -1464,20 +1464,20 @@
             throw OrthancException(ErrorCode_InternalError);
           }
 
-          status = ReadBodyToStream(*stream, connection, headers);
+          postStatus = ReadBodyToStream(*stream, connection, headers);
 
-          if (status == PostDataStatus_Success)
+          if (postStatus == PostDataStatus_Success)
           {
             stream->Execute(output);
           }
         }
         else
         {
-          status = ReadBodyToString(body, connection, headers);
+          postStatus = ReadBodyToString(body, connection, headers);
         }
       }
 
-      switch (status)
+      switch (postStatus)
       {
         case PostDataStatus_NoLength:
           output.SendStatus(HttpStatus_411_LengthRequired);
diff -r f1fe166b21d5 -r 93d2e634fc6d OrthancFramework/Sources/HttpServer/HttpServer.h
--- a/OrthancFramework/Sources/HttpServer/HttpServer.h	Fri Jul 11 17:27:37 2025 +0200
+++ b/OrthancFramework/Sources/HttpServer/HttpServer.h	Fri Jul 11 17:33:28 2025 +0200
@@ -124,7 +124,7 @@
     bool IsRunning() const;
 
   public:
-    HttpServer(MetricsRegistry& metricsRegistry);
+    explicit HttpServer(MetricsRegistry& metricsRegistry);
 
     ~HttpServer();
 
diff -r f1fe166b21d5 -r 93d2e634fc6d OrthancServer/Resources/RunCppCheck-2.17.0.sh
--- a/OrthancServer/Resources/RunCppCheck-2.17.0.sh	Fri Jul 11 17:27:37 2025 +0200
+++ b/OrthancServer/Resources/RunCppCheck-2.17.0.sh	Fri Jul 11 17:33:28 2025 +0200
@@ -9,7 +9,7 @@
 fi
 
 cat <<EOF > /tmp/cppcheck-suppressions.txt
-nullPointer:../../OrthancFramework/UnitTestsSources/RestApiTests.cpp:321
+nullPointer:../../OrthancFramework/UnitTestsSources/RestApiTests.cpp:322
 stlFindInsert:../../OrthancFramework/Sources/DicomFormat/DicomMap.cpp:1525
 stlFindInsert:../../OrthancFramework/Sources/RestApi/RestApiCallDocumentation.cpp:166
 stlFindInsert:../../OrthancFramework/Sources/RestApi/RestApiCallDocumentation.cpp:74
diff -r f1fe166b21d5 -r 93d2e634fc6d OrthancServer/Resources/RunCppCheck.sh
--- a/OrthancServer/Resources/RunCppCheck.sh	Fri Jul 11 17:27:37 2025 +0200
+++ b/OrthancServer/Resources/RunCppCheck.sh	Fri Jul 11 17:33:28 2025 +0200
@@ -12,7 +12,7 @@
 constParameter:../../OrthancFramework/Sources/DicomParsing/FromDcmtkBridge.cpp
 knownArgument:../../OrthancFramework/UnitTestsSources/ImageTests.cpp
 knownConditionTrueFalse:../../OrthancServer/Plugins/Engine/OrthancPlugins.cpp
-nullPointer:../../OrthancFramework/UnitTestsSources/RestApiTests.cpp:321
+nullPointer:../../OrthancFramework/UnitTestsSources/RestApiTests.cpp:322
 stlFindInsert:../../OrthancFramework/Sources/DicomFormat/DicomMap.cpp:1525
 stlFindInsert:../../OrthancFramework/Sources/RestApi/RestApiCallDocumentation.cpp:166
 stlFindInsert:../../OrthancFramework/Sources/RestApi/RestApiCallDocumentation.cpp:74