Split up variants of _native_gvl_profiling_hook_active to fix unuse…

…d argument warning
DataDog · Sep 18, 2024 · ea2bb1b · ea2bb1b
1 parent 567a98c
commit ea2bb1b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/ext/datadog_profiling_native_extension/collectors_cpu_and_wall_time_worker.c b/ext/datadog_profiling_native_extension/collectors_cpu_and_wall_time_worker.c
@@ -1339,15 +1339,15 @@ static VALUE _native_resume_signals(DDTRACE_UNUSED VALUE self) {
 
     state->during_sample = false;
   }
-#endif
 
-static VALUE _native_gvl_profiling_hook_active(DDTRACE_UNUSED VALUE self, VALUE instance) {
-  #ifndef NO_GVL_INSTRUMENTATION
+  static VALUE _native_gvl_profiling_hook_active(DDTRACE_UNUSED VALUE self, VALUE instance) {
     struct cpu_and_wall_time_worker_state *state;
     TypedData_Get_Struct(instance, struct cpu_and_wall_time_worker_state, &cpu_and_wall_time_worker_typed_data, state);
 
     return state->gvl_profiling_hook != NULL ? Qtrue : Qfalse;
-  #else
+  }
+#else
+  static VALUE _native_gvl_profiling_hook_active(DDTRACE_UNUSED VALUE self, DDTRACE_UNUSED VALUE instance) {
     return Qfalse;
-  #endif
-}
+  }
+#endif