Follow-up Comment #2, bug #67992 (group groff):

[comment #1 comment #1:]
> [comment #0 original submission:]
>> $ printf '\\X"pdf: xrev"' | ./build/test-groff -me -a
> 
> This one, at least, is a regression: 

No.

> this pipeline produces no diagnostics on any groff from 1.19.2 to 1.23.0.
> They all produce the desired "simple, pleasant empty document."

It's not a regression to issue a diagnostic when encountering an ill-formed
input document when no diagnostic was issued before.

Failure to diagnose invalid input does not render that input valid.

I need you and Ingo Schwarze to understand that "behavior change"  is not
synonymous with "regression".

Wikipedia:

"Regression testing (rarely, non-regression testing[1]) is re-running
functional and non-functional tests to ensure that previously developed and
tested software still performs as expected after a change."

You had no valid expectation that:

> $ printf '\\X"pdf: xrev"' | ./build/test-groff -me -a

would produce a "simple, pleasant empty document."

Quoting (presumably) Lesk in _ms_(7):


Many
.I nroff
and
.I troff
requests are unsafe in conjunction with
this package, however these requests may be used with
impunity after the first .PP:


I'll grant that the foregoing language is more casual than it could have
been.

* groff's `device` request didn't exist yet (and wouldn't for about another
decade).
* The `\X` escape sequence didn't, either, and Lesk didn't mention escape
sequences as universally unstimulating of punishment, though he probably
should have.
* An `LP` macro call would work just as well as `PP` to obtain the "impunity"
one might seek.

While the _me_ package isn't the same as _ms_, it covers many of the same
bases and appears to be similarly designed.  For evidence, see the "copious"
examples in comment #0.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?67992>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/

Attachment: signature.asc
Description: PGP signature

Reply via email to