Skip to content

Clean up compiler warnings #60674

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 8, 2025
Merged

Conversation

WillAyd
Copy link
Member

@WillAyd WillAyd commented Jan 7, 2025

Not sure why these aren't getting caught in CI, but they show up locally

  [56/152] Compiling C object pandas/_libs/tslibs/conversion.cpython-310-x86_64-linux-gnu.so.p/meson-generated_pandas__libs_tslibs_conversion.pyx.c.o
  In function '__pyx_pf_6pandas_5_libs_6tslibs_10conversion_cast_from_unit_vectorized',
      inlined from '__pyx_pw_6pandas_5_libs_6tslibs_10conversion_1cast_from_unit_vectorized' at pandas/_libs/tslibs/conversion.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/tslibs/conversion.pyx.c:23647:13:
  pandas/_libs/tslibs/conversion.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/tslibs/conversion.pyx.c:3055:79: warning: '__pyx_v_i' may be used uninitialized [-Wmaybe-uninitialized]
   3054 |     (__Pyx_fits_Py_ssize_t(i, type, is_signed) ?\
        |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   3055 |     __Pyx_GetItemInt_Fast(o, (Py_ssize_t)i, is_list, wraparound, boundscheck) :\
        |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
   3056 |     (is_list ? (PyErr_SetString(PyExc_IndexError, "list index out of range"), (PyObject*)NULL) :\
        |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   3057 |                __Pyx_GetItemInt_Generic(o, to_py_func(i))))
        |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  pandas/_libs/tslibs/conversion.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/tslibs/conversion.pyx.c:24573:22: note: in expansion of macro '__Pyx_GetItemInt'
  24573 |         __pyx_t_29 = __Pyx_GetItemInt(((PyObject *)__pyx_v_values), __pyx_v_i, Py_ssize_t, 1, PyInt_FromSsize_t, 0, 0, 0); if (unlikely(!__pyx_t_29)) __PYX_ERR(0, 162, __pyx_L23_error)
        |                      ^~~~~~~~~~~~~~~~
  pandas/_libs/tslibs/conversion.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/tslibs/conversion.pyx.c: In function '__pyx_pw_6pandas_5_libs_6tslibs_10conversion_1cast_from_unit_vectorized':
  pandas/_libs/tslibs/conversion.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/tslibs/conversion.pyx.c:23669:14: note: '__pyx_v_i' was declared here
  23669 |   Py_ssize_t __pyx_v_i;
[87/152] Compiling C object pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/meson-generated_pandas__libs_byteswap.pyx.c.o
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c: In function '__pyx_pf_6pandas_5_libs_8byteswap_read_float_with_byteswap':
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c:2477:61: warning: comparison of integer expressions of different signedness: 'long unsigned int' and 'Py_ssize_t' {aka 'long int'} [-Wsign-compare]
   2477 |     __pyx_t_2 = ((__pyx_v_offset + (sizeof(__pyx_v_value))) < __pyx_t_1);
        |                                                             ^
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c: In function '__pyx_pf_6pandas_5_libs_8byteswap_2read_double_with_byteswap':
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c:2747:61: warning: comparison of integer expressions of different signedness: 'long unsigned int' and 'Py_ssize_t' {aka 'long int'} [-Wsign-compare]
   2747 |     __pyx_t_2 = ((__pyx_v_offset + (sizeof(__pyx_v_value))) < __pyx_t_1);
        |                                                             ^
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c: In function '__pyx_pf_6pandas_5_libs_8byteswap_4read_uint16_with_byteswap':
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c:3015:59: warning: comparison of integer expressions of different signedness: 'long unsigned int' and 'Py_ssize_t' {aka 'long int'} [-Wsign-compare]
   3015 |     __pyx_t_2 = ((__pyx_v_offset + (sizeof(__pyx_v_res))) < __pyx_t_1);
        |                                                           ^
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c: In function '__pyx_pf_6pandas_5_libs_8byteswap_6read_uint32_with_byteswap':
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c:3265:59: warning: comparison of integer expressions of different signedness: 'long unsigned int' and 'Py_ssize_t' {aka 'long int'} [-Wsign-compare]
   3265 |     __pyx_t_2 = ((__pyx_v_offset + (sizeof(__pyx_v_res))) < __pyx_t_1);
        |                                                           ^
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c: In function '__pyx_pf_6pandas_5_libs_8byteswap_8read_uint64_with_byteswap':
  pandas/_libs/byteswap.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/byteswap.pyx.c:3515:59: warning: comparison of integer expressions of different signedness: 'long unsigned int' and 'Py_ssize_t' {aka 'long int'} [-Wsign-compare]
   3515 |     __pyx_t_2 = ((__pyx_v_offset + (sizeof(__pyx_v_res))) < __pyx_t_1);
  [99/152] Compiling C object pandas/_libs/parsers.cpython-310-x86_64-linux-gnu.so.p/meson-generated_pandas__libs_parsers.pyx.c.o
  In file included from pandas/_libs/parsers.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/parsers.pyx.c:1236:
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h: In function 'traced_realloc':
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h:40:7: warning: pointer 'old_ptr' may be used after 'realloc' [-Wuse-after-free]
     40 |       PyTraceMalloc_Untrack(KHASH_TRACE_DOMAIN, (uintptr_t)old_ptr);
        |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h:37:15: note: call to 'realloc' here
     37 |   void *ptr = realloc(old_ptr, size);
  [147/152] Compiling C object pandas/_libs/algos.cpython-310-x86_64-linux-gnu.so.p/meson-generated_pandas__libs_algos.pyx.c.o
  In file included from pandas/_libs/algos.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/algos.pyx.c:1236:
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h: In function 'traced_realloc':
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h:40:7: warning: pointer 'old_ptr' may be used after 'realloc' [-Wuse-after-free]
     40 |       PyTraceMalloc_Untrack(KHASH_TRACE_DOMAIN, (uintptr_t)old_ptr);
        |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h:37:15: note: call to 'realloc' here
     37 |   void *ptr = realloc(old_ptr, size);
        |               ^~~~~~~~~~~~~~~~~~~~~~
  [149/152] Compiling C object pandas/_libs/hashtable.cpython-310-x86_64-linux-gnu.so.p/meson-generated_pandas__libs_hashtable.pyx.c.o
  In file included from pandas/_libs/hashtable.cpython-310-x86_64-linux-gnu.so.p/pandas/_libs/hashtable.pyx.c:1232:
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h: In function 'traced_realloc':
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h:40:7: warning: pointer 'old_ptr' may be used after 'realloc' [-Wuse-after-free]
     40 |       PyTraceMalloc_Untrack(KHASH_TRACE_DOMAIN, (uintptr_t)old_ptr);
        |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  ../pandas/_libs/include/pandas/vendored/klib/khash_python.h:37:15: note: call to 'realloc' here
     37 |   void *ptr = realloc(old_ptr, size);
        |               ^~~~~~~~~~~~~~~~~~~~~~

There's an additional one about some memory view code not being used which probably has to do with Cython, but I haven't looked too far into that

@WillAyd WillAyd requested a review from MarcoGorelli as a code owner January 7, 2025 20:56
@mroeschke mroeschke added Build Library building on various platforms Internals Related to non-user accessible pandas implementation labels Jan 7, 2025
@WillAyd
Copy link
Member Author

WillAyd commented Jan 7, 2025

Not sure the build failures related. Maybe we need to update our CI runners to at least maco13?

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15. For more details, see https://github.com/actions/runner-images/issues/10721

@WillAyd WillAyd force-pushed the fix-compiler-warnings branch from 9586fc1 to 4502660 Compare January 8, 2025 17:37
@mroeschke mroeschke added this to the 3.0 milestone Jan 8, 2025
@mroeschke mroeschke merged commit 8e49cbe into pandas-dev:main Jan 8, 2025
97 of 98 checks passed
@mroeschke
Copy link
Member

Thanks @WillAyd

@WillAyd WillAyd deleted the fix-compiler-warnings branch January 8, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms Internals Related to non-user accessible pandas implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants