From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp1.rz.uni-karlsruhe.de (Debian-exim@smtp1.rz.uni-karlsruhe.de [129.13.185.217]) by krisdoz.my.domain (8.14.3/8.14.3) with ESMTP id oB9Nj5Xk024414 for ; Thu, 9 Dec 2010 18:45:06 -0500 (EST) Received: from hekate.usta.de (asta-nat.asta.uni-karlsruhe.de [172.22.63.82]) by smtp1.rz.uni-karlsruhe.de with esmtp (Exim 4.63 #1) id 1PQqAa-0004EE-A9; Fri, 10 Dec 2010 00:45:04 +0100 Received: from donnerwolke.usta.de ([172.24.96.3]) by hekate.usta.de with esmtp (Exim 4.72) (envelope-from ) id 1PQqAa-0000xl-8c for tech@mdocml.bsd.lv; Fri, 10 Dec 2010 00:45:04 +0100 Received: from iris.usta.de ([172.24.96.5] helo=usta.de) by donnerwolke.usta.de with esmtp (Exim 4.69) (envelope-from ) id 1PQqAa-0003Rm-67 for tech@mdocml.bsd.lv; Fri, 10 Dec 2010 00:45:04 +0100 Received: from schwarze by usta.de with local (Exim 4.72) (envelope-from ) id 1PQqAa-0005eQ-4y for tech@mdocml.bsd.lv; Fri, 10 Dec 2010 00:45:04 +0100 Date: Fri, 10 Dec 2010 00:45:04 +0100 From: Ingo Schwarze To: tech@mdocml.bsd.lv Subject: Re: roff.c question Message-ID: <20101209234504.GF22831@iris.usta.de> References: <4CF678F0.6020304@bsd.lv> <20101201212834.GA22990@iris.usta.de> <4CF77A2B.6020702@bsd.lv> <4CF79F45.6080105@bsd.lv> <20101202225019.GD12188@iris.usta.de> <20101203233148.GC28384@iris.usta.de> <4CFBAD23.1030000@bsd.lv> X-Mailinglist: mdocml-tech Reply-To: tech@mdocml.bsd.lv MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4CFBAD23.1030000@bsd.lv> User-Agent: Mutt/1.5.21 (2010-09-15) Hi Kristaps, Kristaps Dzonsons wrote on Sun, Dec 05, 2010 at 04:17:55PM +0100: > Ingo wrote: >> 2) We should implement some kind of stack limit >> and just bail out. >> >> OK to commit the following patch, too? > Ingo, this is fine by me---conditional upon (1) making the variable > be part of struct roff Hmmm, that doesn't work, struct roff is defined locally in roff.c, so in main.c it has incomplete type. > and not static, But that part does work, i have just made it a member of struct curparse. > and (2) stating the existence of a limit in roff.7. Done, in all brevity. > I realise this is putting a lot of technical > detail in roff.7 (as per last patch response, too) Well, the roff language is full of obscure technicalities, so we can't help that. Hopefully, manual authors will rarely need information from roff(7) and get away with mdoc(7). > but I'm really, really sick of groff's crappy, undocumented > behaviour and would rather too much than too little. > > Can you pop an entry into the TODO to the extent that more rigorous > checking of looping constructs should be enacted? Done. Yours, Ingo -- To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv