From: schwarze <schwarze@openbsd.org>
Date: Fri, 15 Jul 2016 19:31:53 +0000 (+0000)
Subject: To remove the const qualifier from a pointer to an object - either
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=5f1e3782e95a2312bd510677b0f9656d59ca35ec;p=openbsd

To remove the const qualifier from a pointer to an object - either
because we know it is actually mutable or because we are passing
it to a function that doesn't accept a const object but won't
actually attempt to modify it - simply casting from (const type *)
to (type *) is legal C and clearly expresses the intent.
So get rid of the obfuscating UNCONST macro.
Basic idea discussed with guenther@.
---

diff --git a/usr.bin/mandoc/main.c b/usr.bin/mandoc/main.c
index 34d1ed88179..2bc490207e0 100644
--- a/usr.bin/mandoc/main.c
+++ b/usr.bin/mandoc/main.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: main.c,v 1.175 2016/07/15 18:49:53 schwarze Exp $ */
+/*	$OpenBSD: main.c,v 1.176 2016/07/15 19:31:53 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2012, 2014-2016 Ingo Schwarze <schwarze@openbsd.org>
@@ -927,7 +927,7 @@ woptions(struct curparse *curp, char *arg)
 
 	while (*arg) {
 		o = arg;
-		switch (getsubopt(&arg, UNCONST(toks), &v)) {
+		switch (getsubopt(&arg, (char * const *)toks, &v)) {
 		case 0:
 			curp->wstop = 1;
 			break;
diff --git a/usr.bin/mandoc/main.h b/usr.bin/mandoc/main.h
index ee2cbf1d34a..54fb32dbe64 100644
--- a/usr.bin/mandoc/main.h
+++ b/usr.bin/mandoc/main.h
@@ -1,4 +1,4 @@
-/*	$OpenBSD: main.h,v 1.20 2016/07/08 22:27:58 schwarze Exp $ */
+/*	$OpenBSD: main.h,v 1.21 2016/07/15 19:31:53 schwarze Exp $ */
 /*
  * Copyright (c) 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
@@ -16,8 +16,6 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#define	UNCONST(a)	((void *)(uintptr_t)(const void *)(a))
-
 struct	roff_man;
 struct	manoutput;