pyuno/source/module/pyuno.cxx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
New commits: commit 82ade9ab29425b4c76b1a2073325d1269158dc03 Author: Stephan Bergmann <sberg...@redhat.com> AuthorDate: Fri Oct 23 11:08:03 2020 +0200 Commit: Stephan Bergmann <sberg...@redhat.com> CommitDate: Fri Oct 23 17:16:06 2020 +0200 PyLong_AsLongAndOverflow returns long, and that's fine to use here ...where the goal is to check for a value in the range [SAL_MIN_INT32 .. SAL_MAX_INT32], i.e., [-2^31 .. 2^31 - 1]: While C++17 (via C LONG_MIN, LONG_MAX) only guarantees a range of [-(2^31 - 1) .. 2^31 - 1] for long, C++20 now requires two's complement and a fitting range of at least [-2^31 .. 2^31 - 1]. No need for 0d79d216886a71436e705c93829ed66a33270a9c "long->tools::Long in pyuno..sd" here. Change-Id: I6be60b50acfe5ed798cc8e9e1183c336c8d72059 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104712 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sberg...@redhat.com> diff --git a/pyuno/source/module/pyuno.cxx b/pyuno/source/module/pyuno.cxx index 527bcdbf76b5..25aea6436032 100644 --- a/pyuno/source/module/pyuno.cxx +++ b/pyuno/source/module/pyuno.cxx @@ -24,7 +24,6 @@ #include <rtl/strbuf.hxx> #include <rtl/ustrbuf.hxx> -#include <tools/long.hxx> #include <typelib/typedescription.hxx> @@ -319,7 +318,7 @@ static sal_Int32 lcl_PyNumber_AsSal_Int32( PyObject *pObj ) // Convert Python number to platform long, then check actual value against // bounds of sal_Int32 int nOverflow; - tools::Long nResult = PyLong_AsLongAndOverflow( pObj, &nOverflow ); + long nResult = PyLong_AsLongAndOverflow( pObj, &nOverflow ); if ( nOverflow || nResult > SAL_MAX_INT32 || nResult < SAL_MIN_INT32) { PyErr_SetString( PyExc_IndexError, "Python int too large to convert to UNO long" ); return -1; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits