diff options
author | Ingo Schwarze <schwarze@openbsd.org> | 2019-03-13 18:29:18 +0000 |
---|---|---|
committer | Ingo Schwarze <schwarze@openbsd.org> | 2019-03-13 18:29:18 +0000 |
commit | 72861f3e14542e73d6ffe973dc172e3f9ce1f843 (patch) | |
tree | 6a5bd08e422731779554151949cd7961966dc014 /man_validate.c | |
parent | 7030641da6e071e839134f27adcf3e9576998e5b (diff) | |
download | mandoc-72861f3e14542e73d6ffe973dc172e3f9ce1f843.tar.gz mandoc-72861f3e14542e73d6ffe973dc172e3f9ce1f843.tar.zst mandoc-72861f3e14542e73d6ffe973dc172e3f9ce1f843.zip |
Contrary to what the NetBSD attribute(3) manual page suggests,
using __dead instead of __attribute__((__noreturn__)) actually
hinders portability rather than helping it.
Given that mandoc already uses __attribute__ in several files
and that in the portable version, ./configure already contains
rudimentary support for ignoring it on platforms that do not
support it, use __attribute__ directly.
This is expected to fix build failures that Stephen Gregoratto
<dev at sgregoratto dot me> reported from Arch and Debian Linux.
Diffstat (limited to 'man_validate.c')
-rw-r--r-- | man_validate.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/man_validate.c b/man_validate.c index d4a0311a..7c10ea5f 100644 --- a/man_validate.c +++ b/man_validate.c @@ -1,4 +1,4 @@ -/* $Id: man_validate.c,v 1.147 2019/03/11 13:21:11 schwarze Exp $ */ +/* $Id: man_validate.c,v 1.148 2019/03/13 18:29:18 schwarze Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010, 2012-2018 Ingo Schwarze <schwarze@openbsd.org> @@ -41,7 +41,7 @@ typedef void (*v_check)(CHKARGS); -static __dead void check_abort(CHKARGS); +static void check_abort(CHKARGS) __attribute__((__noreturn__)); static void check_par(CHKARGS); static void check_part(CHKARGS); static void check_root(CHKARGS); @@ -196,7 +196,7 @@ check_root(CHKARGS) "(OpenBSD)" : "(NetBSD)"); } -static __dead void +static void check_abort(CHKARGS) { abort(); |