Re: CHECKSTYLE: Report consecutive .?P (paragraph macros)

2022-02-01 Thread G. Branden Robinson
Hi, Alex! At 2022-01-31T00:52:41+0100, Alejandro Colomar (man-pages) wrote: > Recently you reported to me a style error: > > [ > .PP > .TP > tag > paragraph > ] > > In general, consecutive paragraph marcos are wrong, right? .IP, .PP, > .TP (and I may forget some more). Yes. They may or may no

Re: CHECKSTYLE: Report problems with a non-zero status

2022-02-01 Thread G. Branden Robinson
At 2022-01-31T00:41:09+0100, Alejandro Colomar (man-pages) wrote: > Hi Branden, > > I started using groff(1)'s -rCHECKPATCH=3 in my chessutils game suite > (consisting of exactly 1 game right now :~). See >

Re: CHECKSTYLE: Report problems with a non-zero status

2022-02-01 Thread Alejandro Colomar (man-pages)
Hi Branden, On 2/1/22 20:34, G. Branden Robinson wrote: > I have a recommendation or two for how you should be able to get the > behavior you want. > > Quoting your commit diff above... > > +DEFAULT_GROFFFLAGS := -man > +DEFAULT_GROFFFLAGS += -rCHECKSTYLE=3 > +DEFAULT_GROFFFLAGS += -z > +EXTRA_G

Re: CHECKSTYLE: Report consecutive .?P (paragraph macros)

2022-02-01 Thread Bjarni Ingi Gislason
On Mon, Jan 31, 2022 at 12:52:41AM +0100, Alejandro Colomar (man-pages) wrote: > Hi Branden, > > Recently you reported to me a style error: > > [ > .PP > .TP > tag > paragraph > ] > > In general, consecutive paragraph marcos are wrong, right? .IP, .PP, > .TP (and I may forget some more). > > I

Re: CHECKSTYLE: Report consecutive .?P (paragraph macros)

2022-02-01 Thread Alejandro Colomar (man-pages)
Hi Branden, On 2/1/22 21:22, G. Branden Robinson wrote: > I digress, but because I can already hear fundamentalists sweatily > reaching for their Bibles, I'll append some links to heretical > literature. > > > https://web.archive.org/web/20201001070605/https://symas.com/the-sad-s

Re: CHECKSTYLE: Report problems with a non-zero status

2022-02-01 Thread Alejandro Colomar (man-pages)
Hi Branden, On 2/1/22 20:34, G. Branden Robinson wrote: > That's a reasonable thing to want but it runs against the grain of how > *roff programs generally handle diagnostics and exit statuses. > > I have a recommendation or two for how you should be able to get the > behavior you want. Thanks!

Re: CHECKSTYLE: Report consecutive .?P (paragraph macros)

2022-02-01 Thread Alejandro Colomar (man-pages)
Hi Bjarni, On 2/2/22 01:32, Bjarni Ingi Gislason wrote: > A usable tool is "mandoc -T lint". Irrelevant output is just weeded > out with a filter. I like that one too. It catches .IP followed by .PP. I'll add it; I'll use both tools. And for man-pages(7) too. I'll CC you both when I add it

mandoc warning: line scope broken: RE breaks TP (was Re: CHECKSTYLE: Report consecutive .?P (paragraph macros))

2022-02-01 Thread Alejandro Colomar (man-pages)
Hi Branden, Bjarni, On 2/2/22 01:32, Bjarni Ingi Gislason wrote: > On Mon, Jan 31, 2022 at 12:52:41AM +0100, Alejandro Colomar (man-pages) wrote: >> Hi Branden, >> >> Recently you reported to me a style error: >> >> [ >> .PP >> .TP >> tag >> paragraph >> ] >> >> In general, consecutive paragraph m