projects / platform / upstream / gstreamer.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: cb5d31a) | patch
check: Always mark _ck_assert_failed as noreturn
authorJan Alexander Steffens (heftig) <jan.steffens@ltnglobal.com>
Wed, 22 Jul 2020 10:44:02 +0000 (12:44 +0200)
committerGStreamer Merge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 22 Jul 2020 13:45:54 +0000 (13:45 +0000)
commitcd6bc944f908c90b80d6ccbae79ae58811e796e9
treef39b42772984704a24c62fc5c7aa0c9b82f82d5ctree | snapshot
parentcb5d31a2fbf0fac781ff7912f0d62217aeb121ebcommit | diff
check: Always mark _ck_assert_failed as noreturn

So that we can use `fail` like `g_assert_not_reached`.

The comment is apparently wrong or outdated, as GCC considers it legal
for noreturn-marked functions to return using longjmp.

See the thread at
https://gitlab.freedesktop.org/gstreamer/gst-plugins-ugly/-/merge_requests/59#note_576422

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/573>
libs/gst/check/libcheck/check.c diff | blob | history
libs/gst/check/libcheck/check.h.in diff | blob | history
Domain: Multimedia / Media Common;