make sure the leak checker works

geoarrow · Dec 2, 2023 · ec0f06b · ec0f06b
1 parent 59773f9
commit ec0f06b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -55,6 +55,8 @@ install(
   PATTERN "*.h")
 
 if(GEOARROW_GEOS_BUILD_TESTS)
+  set(MEMORYCHECK_COMMAND_OPTIONS "--leak-check=full --error-exitcode=1")
+
   include(CTest)
 
   if(NOT DEFINED CMAKE_CXX_STANDARD)

diff --git a/src/geoarrow_geos/geoarrow_geos_test.cc b/src/geoarrow_geos/geoarrow_geos_test.cc
@@ -23,7 +23,7 @@ class GEOSCppGeometry {
 
   ~GEOSCppGeometry() {
     if (ptr != nullptr) {
-      GEOSGeom_destroy_r(handle, ptr);
+      // GEOSGeom_destroy_r(handle, ptr);
     }
   }
 };