From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ie0-f178.google.com (mail-ie0-f178.google.com [209.85.223.178]) by krisdoz.my.domain (8.14.5/8.14.5) with ESMTP id sAP1XW60008184 for ; Mon, 24 Nov 2014 20:33:33 -0500 (EST) Received: by mail-ie0-f178.google.com with SMTP id tp5so10157000ieb.9 for ; Mon, 24 Nov 2014 17:33:27 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:mime-version:content-type :content-id:date:message-id; bh=zJO5oUdhvbAtdX5i/R6dQKt/jji0uZD+RVcEyMG1Ckc=; b=WuY+cF96mMCswxEkbSvOY0qaXH1VP+WF2kshCSK5A3pJUtMkjTk00TKVLqh4Kh+eWP pbkpvb85AwjKBWKWA63hU5rgQqDv7HPkeqak+aM96qMJlut/QElJfkiorW+O95hCjeES TgWEbvgXFksJCP5f1EYCOd60ngDnjspHQAkX82WMMXvmjE7VOILCwAJIuRyHBDgqgD9d sBdbVB8wOm8cApLWDdm9+TNQOu3uJS99MRxMRNqLLy61d2pBB/ftTDYJNm8ynjOXO1Gs Xn/cVtQ8HpQY9lddPEa//cSZPFidWXYwPi0RIDpvN16m2vxxy2dp7XxVtegSnV3Ybvcz Ujug== X-Gm-Message-State: ALoCoQkY5w8c6P/Xnpm6B9JLjs3/T2GIee8rjk/mJYQnits1I1EcIj1d5Tly+gDPpMhkE59yej8G X-Received: by 10.43.163.69 with SMTP id mn5mr13921768icc.20.1416879206637; Mon, 24 Nov 2014 17:33:26 -0800 (PST) Received: from cathet.us. (75-161-0-121.albq.qwest.net. [75.161.0.121]) by mx.google.com with ESMTPSA id nu1sm7020431igb.0.2014.11.24.17.33.25 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 24 Nov 2014 17:33:26 -0800 (PST) Received: from cathet.us (localhost [127.0.0.1]); by cathet.us. (OpenSMTPD) with ESMTP id f4b0860e; for ; Mon, 24 Nov 2014 18:33:24 -0700 (MST) From: "Anthony J. Bentley" To: tech@mdocml.bsd.lv Subject: tbl(7) should allow comma-separated options X-Mailinglist: mdocml-tech Reply-To: tech@mdocml.bsd.lv MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <15739.1416879204.1@cathet.us> Date: Mon, 24 Nov 2014 18:33:24 -0700 Message-ID: <31999.1416879204@cathet.us> Hi, The following table (from nsdejavu(1)) displays incorrectly in mandoc. Groff displays it correctly. mandoc doesn't like the comma-separated options. Options after the first one are currently ignored. .TS center,box; l l. foo bar .TE mandoc: foo.7:2:7: ERROR: bad table syntax Here's a patch (but is it right?): Index: tbl_opts.c =================================================================== RCS file: /cvs/src/usr.bin/mandoc/tbl_opts.c,v retrieving revision 1.5 diff -u -p -u -p -r1.5 tbl_opts.c --- tbl_opts.c 20 Apr 2014 16:44:44 -0000 1.5 +++ tbl_opts.c 25 Nov 2014 01:29:42 -0000 @@ -178,7 +178,7 @@ again: /* * * options ::= option_list [:space:]* [;][\n] * option_list ::= option option_tail - * option_tail ::= [:space:]+ option_list | + * option_tail ::= [,:space:]+ option_list | * ::= epsilon * option ::= [:alpha:]+ args * args ::= [:space:]* [(] [:alpha:]+ [)] @@ -209,7 +209,7 @@ again: /* buf[i] = '\0'; - while (isspace((unsigned char)p[*pos])) + while (isspace((unsigned char)p[*pos]) || p[*pos] == ',') (*pos)++; /* -- To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv