exclude un-hittable lines from coverage

P403n1x87 · P403n1x87 · commit e0b1b059cc29 · 2025-05-31T19:28:58.000+01:00
diff --git a/src/austin.c b/src/austin.c
@@ -308,9 +308,9 @@ main(int argc, char** argv) {
     }
 
     if (!isvalid(handler)) {
-        log_e("Failed to create event handler");
+        log_e("Failed to create event handler"); // GCOV_EXCL_START
         retval = -1;
-        goto release;
+        goto release; // GCOV_EXCL_STOP
     }
     event_handler_install(handler);
 
diff --git a/src/events.c b/src/events.c
@@ -98,7 +98,7 @@ collapsed_stack_event_handler__handle_stack_end(base_event_handler_t* self) {
         frame_t* native_frame = stack_native_pop();
         if (!isvalid(native_frame)) {
             log_e("Invalid native frame");
-            break;
+            break; // GCOV_EXCL_LINE
         }
         cached_string_t* scope = native_frame->scope;
 
@@ -166,8 +166,8 @@ event_handler_t*
 collapsed_stack_event_handler_new(void) {
     event_handler_t* handler = (event_handler_t*)calloc(1, sizeof(base_event_handler_t));
     if (!isvalid(handler)) {
-        log_e("Failed to allocate memory for event handler");
-        return NULL;
+        log_e("Failed to allocate memory for event handler"); // GCOV_EXCL_START
+        return NULL;                                          // GCOV_EXCL_STOP
     }
 
     handler->spec.emit_stack_begin = (event_handler_stack_begin_t)collapsed_stack_event_handler__handle_stack_begin;
@@ -230,8 +230,8 @@ mojo_event_handler__handle_stack_end(base_event_handler_t* self) {
     while (!stack_native_is_empty()) {
         frame_t* native_frame = stack_native_pop();
         if (!isvalid(native_frame)) {
-            log_e("Invalid native frame");
-            break;
+            log_e("Invalid native frame"); // GCOV_EXCL_START
+            break;                         // GCOV_EXCL_STOP
         }
         cached_string_t* scope = native_frame->scope;
         int is_frame_eval = (scope == UNKNOWN_SCOPE) ? FALSE : isvalid(strstr(scope->value, "PyEval_EvalFrameDefault"));
@@ -296,8 +296,8 @@ event_handler_t*
 mojo_event_handler_new(void) {
     event_handler_t* handler = (event_handler_t*)calloc(1, sizeof(base_event_handler_t));
     if (!isvalid(handler)) {
-        log_e("Failed to allocate memory for event handler");
-        return NULL;
+        log_e("Failed to allocate memory for event handler"); // GCOV_EXCL_START
+        return NULL;                                          // GCOV_EXCL_STOP
     }
 
     handler->spec.emit_stack_begin = (event_handler_stack_begin_t)mojo_event_handler__handle_stack_begin;
@@ -346,12 +346,12 @@ where_event_handler__handle_stack_end(base_event_handler_t* self) {
     while (!stack_native_is_empty()) {
         frame_t* native_frame = stack_native_pop();
         if (!isvalid(native_frame)) {
-            log_e("Invalid native frame");
-            break;
+            log_e("Invalid native frame"); // GCOV_EXCL_START
+            break;                         // GCOV_EXCL_STOP
         }
         cached_string_t* scope = native_frame->scope;
         if (!isvalid(scope)) {
-            scope = UNKNOWN_SCOPE;
+            scope = UNKNOWN_SCOPE; // GCOV_EXCL_LINE
         }
 
         int is_frame_eval = (scope == UNKNOWN_SCOPE) ? FALSE : isvalid(strstr(scope->value, "PyEval_EvalFrameDefault"));
@@ -396,8 +396,8 @@ event_handler_t*
 where_event_handler_new(void) {
     event_handler_t* handler = (event_handler_t*)calloc(1, sizeof(base_event_handler_t));
     if (!isvalid(handler)) {
-        log_e("Failed to allocate memory for event handler");
-        return NULL;
+        log_e("Failed to allocate memory for event handler"); // GCOV_EXCL_START
+        return NULL;                                          // GCOV_EXCL_STOP
     }
 
     handler->spec.emit_stack_begin = (event_handler_stack_begin_t)where_event_handler__handle_stack_begin;
diff --git a/src/events.h b/src/events.h
@@ -92,7 +92,7 @@ extern
 static inline void
 event_handler__emit_stack_begin(sample_t* sample) {
     if (!isvalid(event_handler))
-        return;
+        return; // GCOV_EXCL_LINE
 
     event_handler_stack_begin_t handler = event_handler->spec.emit_stack_begin;
     if (isvalid(handler))
@@ -102,7 +102,7 @@ event_handler__emit_stack_begin(sample_t* sample) {
 static inline void
 event_handler__emit_metadata(char* key, char* value, ...) {
     if (!isvalid(event_handler))
-        return;
+        return; // GCOV_EXCL_LINE
 
     va_list args;
     va_start(args, value);
@@ -117,7 +117,7 @@ event_handler__emit_metadata(char* key, char* value, ...) {
 static inline void
 event_handler__emit_new_string(cached_string_t* cached_string) {
     if (!isvalid(event_handler))
-        return;
+        return; // GCOV_EXCL_LINE
 
     event_handler_new_string_t handler = event_handler->spec.emit_new_string;
     if (isvalid(handler))
@@ -127,7 +127,7 @@ event_handler__emit_new_string(cached_string_t* cached_string) {
 static inline void
 event_handler__emit_new_frame(void* frame) {
     if (!isvalid(event_handler))
-        return;
+        return; // GCOV_EXCL_LINE
 
     event_handler_new_frame_t handler = event_handler->spec.emit_new_frame;
     if (isvalid(handler))
@@ -137,7 +137,7 @@ event_handler__emit_new_frame(void* frame) {
 static inline void
 event_handler__emit_stack_end(void) {
     if (!isvalid(event_handler))
-        return;
+        return; // GCOV_EXCL_LINE
 
     event_handler_stack_end_t handler = event_handler->spec.emit_stack_end;
     if (isvalid(handler))
diff --git a/src/py_proc.c b/src/py_proc.c
@@ -1036,14 +1036,14 @@ _py_proc__get_memory_delta(py_proc_t* self) {
 int
 py_proc__get_gc_state(py_proc_t* self) {
     if (!isvalid(self->gc_state_raddr))
-        return GC_STATE_UNKNOWN;
+        return GC_STATE_UNKNOWN; // GCOV_EXCL_LINE
 
     V_DESC(self->py_v);
 
     GCRuntimeState gc_state;
     if (fail(py_proc__get_type(self, self->gc_state_raddr, gc_state))) {
         log_d("Failed to get GC runtime state");
-        return GC_STATE_UNKNOWN;
+        return GC_STATE_UNKNOWN; // GCOV_EXCL_LINE
     }
 
     return V_FIELD(int, gc_state, py_gc, o_collecting);
diff --git a/src/py_string.h b/src/py_string.h
@@ -50,7 +50,7 @@ static inline cached_string_t*
 cached_string_new(key_dt key, char* value) {
     cached_string_t* cached_string = (cached_string_t*)malloc(sizeof(cached_string_t));
     if (!isvalid(cached_string)) {
-        return NULL;
+        return NULL; // GCOV_EXCL_LINE
     }
 
     cached_string->key   = key;
diff --git a/src/py_thread.c b/src/py_thread.c
@@ -613,9 +613,9 @@ _py_thread__unwind_native_frame_stack(py_thread_t* self) {
                         if (unw_get_proc_name(&cursor, _native_buf, MAXLEN, &offset) == 0) {
                             scope = cached_string_new(scope_key, strdup(_native_buf));
                             if (!isvalid(scope)) {
-                                log_ie("Failed to create scope string");
+                                log_ie("Failed to create scope string"); // GCOV_EXCL_START
                                 set_error(ETHREAD);
-                                FAIL;
+                                FAIL; // GCOV_EXCL_STOP
                             }
                             lru_cache__store(string_cache, scope_key, (value_t)scope);
                             event_handler__emit_new_string(scope);
@@ -630,9 +630,9 @@ _py_thread__unwind_native_frame_stack(py_thread_t* self) {
                 if (isvalid(range)) { // For now this is only relevant in `where` mode
                     filename = cached_string_new((key_dt)pc, range->name);
                     if (!isvalid(filename)) {
-                        log_ie("Failed to create filename string");
+                        log_ie("Failed to create filename string"); // GCOV_EXCL_START
                         set_error(ETHREAD);
-                        FAIL;
+                        FAIL; // GCOV_EXCL_STOP
                     }
                 } else {
                     // The program counter carries information about the file name *and*
@@ -647,9 +647,9 @@ _py_thread__unwind_native_frame_stack(py_thread_t* self) {
                         sprintf(_native_buf, "native@%" PRIxPTR, pc);
                         filename = cached_string_new(filename_key, strdup(_native_buf));
                         if (!isvalid(filename)) {
-                            log_ie("Failed to create filename string");
+                            log_ie("Failed to create filename string"); // GCOV_EXCL_START
                             set_error(ETHREAD);
-                            FAIL;
+                            FAIL; // GCOV_EXCL_STOP
                         }
                         lru_cache__store(string_cache, filename_key, (value_t)filename);
                         event_handler__emit_new_string(filename);
diff --git a/src/stack.c b/src/stack.c
@@ -39,11 +39,11 @@
 int
 stack_allocate(size_t size) {
     if (isvalid(_stack))
-        SUCCESS;
+        SUCCESS; // GCOV_EXCL_LINE
 
     _stack = (stack_dt*)calloc(1, sizeof(stack_dt));
     if (!isvalid(_stack))
-        FAIL;
+        FAIL; // GCOV_EXCL_LINE
 
     _stack->size    = size;
     _stack->base    = (frame_t**)calloc(size, sizeof(frame_t*));
@@ -59,7 +59,7 @@ stack_allocate(size_t size) {
 void
 stack_deallocate(void) {
     if (!isvalid(_stack))
-        return;
+        return; // GCOV_EXCL_LINE
 
     free(_stack->base);
     free(_stack->py_base);

Original file line number	Diff line number	Diff line change
`@@ -308,9 +308,9 @@ main(int argc, char** argv) {`
`308`	`308`	`}`
`309`	`309`
`310`	`310`	`if (!isvalid(handler)) {`
`311`		`- log_e("Failed to create event handler");`
	`311`	`+ log_e("Failed to create event handler"); // GCOV_EXCL_START`
`312`	`312`	`retval = -1;`
`313`		`- goto release;`
	`313`	`+ goto release; // GCOV_EXCL_STOP`
`314`	`314`	`}`
`315`	`315`	`event_handler_install(handler);`
`316`	`316`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ static inline cached_string_t*`
`50`	`50`	`cached_string_new(key_dt key, char* value) {`
`51`	`51`	`cached_string_t* cached_string = (cached_string_t*)malloc(sizeof(cached_string_t));`
`52`	`52`	`if (!isvalid(cached_string)) {`
`53`		`- return NULL;`
	`53`	`+ return NULL; // GCOV_EXCL_LINE`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`cached_string->key = key;`