GRIDEDIT-1546 Removed some commented out code

Deltares · Nov 26, 2024 · 0bbf445 · 0bbf445
1 parent 544f088
commit 0bbf445
Showing 1 changed file with 9 additions and 10 deletions.
diff --git a/libs/MeshKernelApi/src/MeshKernel.cpp b/libs/MeshKernelApi/src/MeshKernel.cpp
@@ -139,6 +139,7 @@ namespace meshkernelapi
         return propertyMap;
     }
 
+    /// @brief Map of property calculators, from an property identifier to the calculator.
     static std::map<int, std::unique_ptr<PropertyCalculator>> propertyCalculators = allocatePropertyCalculators();
 
     static meshkernel::ExitCode HandleException(std::exception_ptr exception_ptr = std::current_exception())
@@ -1534,10 +1535,6 @@ namespace meshkernelapi
             {
                 propertyCalculators[propertyValue]->Calculate(meshKernelState.at(meshKernelId), geometryList);
             }
-            // if (propertyCalculators.contains(propertyValue) && propertyCalculators[propertyValue] != nullptr)
-            // {
-            //     propertyCalculators[propertyValue]->Calculate(meshKernelId, geometryList);
-            // }
             else
             {
                 throw meshkernel::MeshKernelError("Property not supported");
@@ -1553,6 +1550,8 @@ namespace meshkernelapi
     MKERNEL_API int mkernel_mesh2d_get_property_dimension(int meshKernelId, int propertyValue, int& dimension)
     {
         lastExitCode = meshkernel::ExitCode::Success;
+        dimension = -1;
+
         try
         {
             if (!meshKernelState.contains(meshKernelId))
@@ -1567,8 +1566,6 @@ namespace meshkernelapi
                 return lastExitCode;
             }
 
-            dimension = -1;
-
             if (propertyCalculators.contains(propertyValue) && propertyCalculators[propertyValue] != nullptr)
             {
                 dimension = propertyCalculators[propertyValue]->Size(meshKernelState.at(meshKernelId));
@@ -1602,10 +1599,12 @@ namespace meshkernelapi
 
             const auto result = meshKernelState[meshKernelId].m_mesh2d->GetSmoothness();
 
-            for (auto i = 0; i < geometryList.num_coordinates; ++i)
-            {
-                geometryList.values[i] = result[i];
-            }
+            std::copy (result.begin (), result.end (), geometryList.values);
+
+            // for (auto i = 0; i < geometryList.num_coordinates; ++i)
+            // {
+            //     geometryList.values[i] = result[i];
+            // }
         }
         catch (...)
         {