Merge pull request #152 from wgoadri/fix-max-node-count

[SpatialPartitioning] Force max node count type to 'std::size_t'
poncateam · Oct 29, 2024 · aa50bfd · aa50bfd
2 parents 045d6c2 + 865a7c3
commit aa50bfd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG b/CHANGELOG
@@ -9,6 +9,9 @@ Current head (v.1.4 RC)
     - [build] Enable exportation of projects linking to Ponca targets (#150)
     - [install] Change output directory to `lib/cmake/Ponca` (#150)
 
+- Bug-fixes and code improvements
+   - [spatialPartitioning] Fix compilation error with `MAX_NODE_COUNT` when compiled with MSVC (#152)
+
 --------------------------------------------------------------------------------
 v.1.3
 This release introduces several improvements around the KdTre API, as well as bug fixes, new features and doc

diff --git a/Ponca/src/SpatialPartitioning/KdTree/kdTreeTraits.h b/Ponca/src/SpatialPartitioning/KdTree/kdTreeTraits.h
@@ -114,7 +114,7 @@ class KdTreeCustomizableNode
     using LeafType  = _LeafNodeType;
 
 public:
-    enum
+    enum : std::size_t
     {
         /*!
          * \brief The maximum number of nodes that a kd-tree can have when using
-Original file line number
+Diff line change
@@ Expand Up / @@ -114,7 +114,7 @@ class KdTreeCustomizableNode @@
         using LeafType  = _LeafNodeType;
     public:
-        enum
+        enum : std::size_t
         {
             /*!
              * \brief The maximum number of nodes that a kd-tree can have when using
@@ Expand Down @@