aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/mdoc_validate.c
diff options
context:
space:
mode:
authorKristaps Dzonsons <kristaps@bsd.lv>2010-06-12 11:41:50 +0000
committerKristaps Dzonsons <kristaps@bsd.lv>2010-06-12 11:41:50 +0000
commit9e1c8c4504be3ee8555871134f21b59fba7bd8ac (patch)
treef962a1dc9d54009b312c3de97f2013da79ae5c9f /mdoc_validate.c
parentaf1a16ee46c535e7e97fe26fc4c6c05844de41e2 (diff)
downloadmandoc-9e1c8c4504be3ee8555871134f21b59fba7bd8ac.tar.gz
mandoc-9e1c8c4504be3ee8555871134f21b59fba7bd8ac.tar.zst
mandoc-9e1c8c4504be3ee8555871134f21b59fba7bd8ac.zip
Removed stipulation that an empty `Bd -offset' will default to 6n. Not
sure where this came about. Added regression tests to convince myself that this is so. Also consolidated COMPATIBILITY notes regarding `Bd'. Added COMPATIBILITY note to the effect that old groff pukes on `Bd -compact -ragged' (regression test will fail on old groff).
Diffstat (limited to 'mdoc_validate.c')
-rw-r--r--mdoc_validate.c21
1 files changed, 3 insertions, 18 deletions
diff --git a/mdoc_validate.c b/mdoc_validate.c
index 59e3d79c..ad7b6418 100644
--- a/mdoc_validate.c
+++ b/mdoc_validate.c
@@ -1,4 +1,4 @@
-/* $Id: mdoc_validate.c,v 1.94 2010/06/12 11:21:44 kristaps Exp $ */
+/* $Id: mdoc_validate.c,v 1.95 2010/06/12 11:41:50 kristaps Exp $ */
/*
* Copyright (c) 2008, 2009 Kristaps Dzonsons <kristaps@bsd.lv>
*
@@ -731,23 +731,8 @@ pre_bd(PRE_ARGS)
dup = (NULL != n->data.Bd.offs);
break;
}
- /*
- * If empty, assign it to a sane default, which
- * groff stipulates is about 8n.
- */
- /*
- * FIXME: remove this.
- *
- * Where the hell did I get the idea that this
- * happens?
- */
- assert(1 == n->args->refcnt);
- n->args->argv[i].sz++;
- n->args->argv[i].value =
- mandoc_malloc(sizeof(char *));
- n->args->argv[i].value[0] =
- mandoc_strdup("8n");
- offs = n->args->argv[i].value[0];
+ if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_IGNARGV))
+ return(0);
break;
case (MDOC_Compact):
comp = 1;