Re: [Groff] bad QE in -ms

2016-01-31 Thread Carsten Kunze
The paragraph spacing bug is also visible in the re-typeset book utp_book-1.1.pdf (page 90). The original book does have vertical space before the quote (page 110). The sentence "Use the .QP macro inside a .QS/.QE block to break up paragraphs." on this page does IMHO has a typo--.LP or .PP should

Re: [Groff] bad QE in -ms

2016-01-31 Thread Carsten Kunze
Doug McIlroy wrote: > The -ms macro QE typically draws a misleading diagnostic, "unbalanced QE". > This breaks old documents that expect QP-QE (quoted paragraph) to narrow > the text and then restore it. The groff QE does not restore. I had been wrong here. For the simple tests I've done groff

Re: [Groff] bad QE in -ms

2016-01-31 Thread Doug McIlroy
I found several old copies of tmac.s lying around in my files, presumably modified for various special purposes. They all contain not only QP-QE, but also an undocumented QS, and a never-tested flag set by QP and unset by LP and PP. It appears that QS was expected to be used when QE is not immediat

Re: [Groff] bad QE in -ms

2016-01-31 Thread Larry Kollar
> Carsten Kunze wrote: > > Doug McIlroy wrote: >> Bad case I: [...] > > It neither works in groff nor in heirloom roff as specified (in original -ms > specification). > > I suggest to make .QS, .QE, .QP work as specified for traditional -ms without > any warnings. Then there may be some re

Re: [Groff] bad QE in -ms

2016-01-31 Thread Werner LEMBERG
> It neither works in groff nor in heirloom roff as specified (in > original -ms specification). [...] Carsten, you now have write access to the groff repository, and the copyright papers arrived at the FSF also, so please do whatever seems to fit :-) Werner

Re: [Groff] bad QE in -ms

2016-01-31 Thread Carsten Kunze
Doug McIlroy wrote: > Bad case I: [...] It neither works in groff nor in heirloom roff as specified (in original -ms specification). I suggest to make .QS, .QE, .QP work as specified for traditional -ms without any warnings. Then there may be some redundant code in .QE and .RE but this shoul