aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/man_validate.c
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@openbsd.org>2019-03-13 18:29:18 +0000
committerIngo Schwarze <schwarze@openbsd.org>2019-03-13 18:29:18 +0000
commit72861f3e14542e73d6ffe973dc172e3f9ce1f843 (patch)
tree6a5bd08e422731779554151949cd7961966dc014 /man_validate.c
parent7030641da6e071e839134f27adcf3e9576998e5b (diff)
downloadmandoc-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.c6
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();