Re: mdocml: Bail out explicitly on invalid .Rs content like: .Rs plain text

[Ingo Schwarze <schwarze@usta.de>]

Hi Joerg,

Joerg Sonnenberger wrote on Sun, Nov 20, 2011 at 05:27:41AM +0100:
> On Sat, Nov 19, 2011 at 02:32:20PM +0100, Ingo Schwarze wrote:

>> i could confirm both the problem and the fix, and i don't see any
>> regressions, so i have merged this to OpenBSD and improved the
>> comment a bit.

> There is one aspect to consider. We drop
> .Rs
> plain text
> .Re
> silently now, when groff renders it.

Sure, i have a test case for that in my private collection of
failing test cases.  If anybody considers it of sufficient
importance, feel free to add it to TODO.  Typically, i add
stuff to TODO when i see a case in practice where it matters,
and i haven't yet seen one involving plain text in .Rs.

ischwarze@isnote $ cd /usr/src/regress/usr.bin/mandoc/TODO/                    
ischwarze@isnote $ find . -name '*.in' | wc -l              
      98

For the time being, your patch prevents a crash, which is already
a real improvement.

Yours,
  Ingo
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv