aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/compat_getsubopt.c
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@openbsd.org>2014-08-17 20:53:50 +0000
committerIngo Schwarze <schwarze@openbsd.org>2014-08-17 20:53:50 +0000
commit6bc581d1a26667c00bfa4397c833b05ca718ee9a (patch)
tree74339035f630d5731c5b26bf1632e1e90c75b60e /compat_getsubopt.c
parent6dae70845e149f75383a1bbb4b745deffd14632e (diff)
downloadmandoc-6bc581d1a26667c00bfa4397c833b05ca718ee9a.tar.gz
mandoc-6bc581d1a26667c00bfa4397c833b05ca718ee9a.tar.zst
mandoc-6bc581d1a26667c00bfa4397c833b05ca718ee9a.zip
Do not require getsubopt() to provide extern char *suboptarg.
We don't use it anyway in mandoc. Like this, fewer systems need the compat implementation. In particular, we can now use the stock getsubopt() on glibc and musl. Besides, the comment in the BSD getsubopt.c that error messages are tricky without *suboptarg is massively overblown. If you simply save a copy of the pointer you pass into getsubopt(), that's quite usable for an error message. People start campaigning for the addition of *suboptarg to C libraries on the grounds that mandoc wants it, but actually, i consider library functions manipulating global data quite ugly, so stop pushing people into that questionable direction. While here, add an explicit Copyright header to the test file. While it's obviously to me what Kristaps intended, others might consider this file copyrightable and wonder what's up.
Diffstat (limited to 'compat_getsubopt.c')
-rw-r--r--compat_getsubopt.c10
1 files changed, 2 insertions, 8 deletions
diff --git a/compat_getsubopt.c b/compat_getsubopt.c
index 3a4a7d4a..880f2f78 100644
--- a/compat_getsubopt.c
+++ b/compat_getsubopt.c
@@ -6,6 +6,7 @@ int dummy;
#else
+/* $Id: compat_getsubopt.c,v 1.5 2014/08/17 20:53:50 schwarze Exp $ */
/* $OpenBSD: getsubopt.c,v 1.4 2005/08/08 08:05:36 espie Exp $ */
/*-
@@ -41,18 +42,11 @@ int dummy;
#include <stdlib.h>
#include <string.h>
-/*
- * The SVID interface to getsubopt provides no way of figuring out which
- * part of the suboptions list wasn't matched. This makes error messages
- * tricky... The extern variable suboptarg is a pointer to the token
- * which didn't match.
- */
-char *suboptarg;
-
int
getsubopt(char **optionp, char * const *tokens, char **valuep)
{
int cnt;
+ char *suboptarg;
char *p;
suboptarg = *valuep = NULL;