Merge pull request #2102 from jcarpent/topic/python

Use bp::ssize_t for recent version of Windows compiler
stack-of-tasks · Nov 28, 2023 · 0c12ba5 · 0c12ba5
2 parents 06be82d + bd23aaf
commit 0c12ba5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,7 +8,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### Fixed
 
-- Add missing include `<sys/types.h>` for recent version of Windows compilers ([#2101](https://github.com/stack-of-tasks/pinocchio/pull/2101))
+- Use bp::ssize_t for recent version of Windows compilers ([#2102](https://github.com/stack-of-tasks/pinocchio/pull/2102))
 
 ## [2.6.21] - 2023-11-27
 

diff --git a/bindings/python/utils/conversions.cpp b/bindings/python/utils/conversions.cpp
@@ -3,7 +3,6 @@
 //
 
 #include <sstream>
-#include <sys/types.h>
 
 #include "pinocchio/bindings/python/fwd.hpp"
 #include "pinocchio/bindings/python/spatial/se3.hpp"
@@ -38,7 +37,7 @@ namespace pinocchio
     template <typename TupleOrList>
     SE3 XYZQUATToSE3_bp(const TupleOrList& v)
     {
-      ssize_t size = bp::len(v);
+      bp::ssize_t size = bp::len(v);
       if(size != 7)
       {
         throw std::invalid_argument(