From 8e49cbef009982c296e6cbaa3eb5cdc5bf5e89f6 Mon Sep 17 00:00:00 2001
From: William Ayd <will_ayd@innobi.io>
Date: Wed, 8 Jan 2025 14:08:40 -0500
Subject: [PATCH] Clean up compiler warnings (#60674)

---
 pandas/_libs/byteswap.pyx                        | 10 +++++-----
 .../include/pandas/vendored/klib/khash_python.h  |  4 +---
 pandas/_libs/tslibs/conversion.pyx               | 16 ++++++++--------
 3 files changed, 14 insertions(+), 16 deletions(-)

diff --git a/pandas/_libs/byteswap.pyx b/pandas/_libs/byteswap.pyx
index 67cd7ad58d229..7a8a9fc5a9139 100644
--- a/pandas/_libs/byteswap.pyx
+++ b/pandas/_libs/byteswap.pyx
@@ -15,7 +15,7 @@ from libc.string cimport memcpy
 
 def read_float_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
     cdef uint32_t value
-    assert offset + sizeof(value) < len(data)
+    assert offset + <Py_ssize_t>sizeof(value) < len(data)
     cdef const void *ptr = <unsigned char*>(data) + offset
     memcpy(&value, ptr, sizeof(value))
     if byteswap:
@@ -28,7 +28,7 @@ def read_float_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
 
 def read_double_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
     cdef uint64_t value
-    assert offset + sizeof(value) < len(data)
+    assert offset + <Py_ssize_t>sizeof(value) < len(data)
     cdef const void *ptr = <unsigned char*>(data) + offset
     memcpy(&value, ptr, sizeof(value))
     if byteswap:
@@ -41,7 +41,7 @@ def read_double_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
 
 def read_uint16_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
     cdef uint16_t res
-    assert offset + sizeof(res) < len(data)
+    assert offset + <Py_ssize_t>sizeof(res) < len(data)
     memcpy(&res, <const unsigned char*>(data) + offset, sizeof(res))
     if byteswap:
         res = _byteswap2(res)
@@ -50,7 +50,7 @@ def read_uint16_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
 
 def read_uint32_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
     cdef uint32_t res
-    assert offset + sizeof(res) < len(data)
+    assert offset + <Py_ssize_t>sizeof(res) < len(data)
     memcpy(&res, <const unsigned char*>(data) + offset, sizeof(res))
     if byteswap:
         res = _byteswap4(res)
@@ -59,7 +59,7 @@ def read_uint32_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
 
 def read_uint64_with_byteswap(bytes data, Py_ssize_t offset, bint byteswap):
     cdef uint64_t res
-    assert offset + sizeof(res) < len(data)
+    assert offset + <Py_ssize_t>sizeof(res) < len(data)
     memcpy(&res, <const unsigned char*>(data) + offset, sizeof(res))
     if byteswap:
         res = _byteswap8(res)
diff --git a/pandas/_libs/include/pandas/vendored/klib/khash_python.h b/pandas/_libs/include/pandas/vendored/klib/khash_python.h
index 9706a8211b61f..04e2d1e39ce2a 100644
--- a/pandas/_libs/include/pandas/vendored/klib/khash_python.h
+++ b/pandas/_libs/include/pandas/vendored/klib/khash_python.h
@@ -34,11 +34,9 @@ static void *traced_calloc(size_t num, size_t size) {
 }
 
 static void *traced_realloc(void *old_ptr, size_t size) {
+  PyTraceMalloc_Untrack(KHASH_TRACE_DOMAIN, (uintptr_t)old_ptr);
   void *ptr = realloc(old_ptr, size);
   if (ptr != NULL) {
-    if (old_ptr != ptr) {
-      PyTraceMalloc_Untrack(KHASH_TRACE_DOMAIN, (uintptr_t)old_ptr);
-    }
     PyTraceMalloc_Track(KHASH_TRACE_DOMAIN, (uintptr_t)ptr, size);
   }
   return ptr;
diff --git a/pandas/_libs/tslibs/conversion.pyx b/pandas/_libs/tslibs/conversion.pyx
index a635dd33f8420..7a8b4df447aee 100644
--- a/pandas/_libs/tslibs/conversion.pyx
+++ b/pandas/_libs/tslibs/conversion.pyx
@@ -149,18 +149,18 @@ def cast_from_unit_vectorized(
     if p:
         frac = np.round(frac, p)
 
-    try:
-        for i in range(len(values)):
+    for i in range(len(values)):
+        try:
             if base[i] == NPY_NAT:
                 out[i] = NPY_NAT
             else:
                 out[i] = <int64_t>(base[i] * m) + <int64_t>(frac[i] * m)
-    except (OverflowError, FloatingPointError) as err:
-        # FloatingPointError can be issued if we have float dtype and have
-        #  set np.errstate(over="raise")
-        raise OutOfBoundsDatetime(
-            f"cannot convert input {values[i]} with the unit '{unit}'"
-        ) from err
+        except (OverflowError, FloatingPointError) as err:
+            # FloatingPointError can be issued if we have float dtype and have
+            #  set np.errstate(over="raise")
+            raise OutOfBoundsDatetime(
+                f"cannot convert input {values[i]} with the unit '{unit}'"
+            ) from err
     return out