Re: tweak message "scope open on exit"

[Kristaps Dzonsons <kristaps@bsd.lv>]

> in mdoc(7), if an explicit scope is still open at the end of an input
> file, that's FATAL.  I really don't see why, it's trivial to just
> close out the block and proceed to rendering the file.
> 
> On the other hand, in man(7), if an explicit scope is still open at
> the end of an input file, that's merely a WARNING.  I regard that
> as dangerous, because an unclosed block probably means that either:
> 
>  - some input text is missing, resulting in incomplete output
>  - or closure of a block is missing, probably resulting in
>    garbled document structure from that point onwards.
> 
> As i see it, that's a typical case of a non-fatal ERROR:
> We can render, but probably infomation is missing or the
> structure is seriously mangled.
> 
> Besides, why not use the same error type for mdoc(7) and man(7)?
> 
> And finally, since we do know that the error is non-fatal,
> there is no need to check the return value of the msg function.

Ingo, this looks good.  At first I dithered for a while about returning 
0 to mdoc_endparse(), but because the error is non-fatal, your reasoning 
is sound and I agree.

Can you fix up the WARNING->ERROR logic in libman, too?

Thanks,

Kristaps
--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv