[vlc-devel] [PATCH] lib: assert instead of abort
Thomas Guillem
thomas at gllm.fr
Fri Sep 20 11:21:37 CEST 2019
Because you should not abort in production. Developers will still be notified
that they are miss-using the API with the assert.
PS: this abort was happening on iOS via VLCKit, the API miss-use had been fixed
here: https://code.videolan.org/videolan/VLCKit/merge_requests/19
---
lib/event.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/event.c b/lib/event.c
index 28d3d0c232..4a57029b89 100644
--- a/lib/event.c
+++ b/lib/event.c
@@ -179,5 +179,5 @@ void libvlc_event_detach(libvlc_event_manager_t *em, libvlc_event_type_t type,
return;
}
}
- abort();
+ assert(!"event not found or already detached");
}
--
2.20.1
More information about the vlc-devel
mailing list