Update of bug #66686 (group groff):

                  Status:                    None => Confirmed
             Assigned to:                    None => gbranden

    _______________________________________________________

Follow-up Comment #3:

Found it.


1e53037d5d3f934cf89081436f39f87ff2f2efad is the first bad commit
commit 1e53037d5d3f934cf89081436f39f87ff2f2efad
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Mon Aug 19 14:11:56 2024 -0500

    [troff]: Fix Savannah #63142 (newlines as delims).
    
    Withdraw support for newlines as escape sequence delimiters.  Only a few
    (6) supported this, and the rationale behind it is unknown.  DWB 3.3
    troff didn't behave this way in any case.
    
    * src/roff/troff/input.cpp (do_overstrike, do_bracket, do_name_test)
      (do_zero_width_output, do_width, do_device_control): Do it.  As a
      bonus, check starting delimters for these escape sequences
      (`\[obAZwX]`) for validity in general.
    
      (do_overstrike, do_bracket, do_zero_width_output): When returning
      early, delete the `new` node we just allocated to avoid leaking
      memory.
    
      (token:next): Be prepared for the `\b` and `\o` escape sequences
      to return a null pointer (if the sequence doesn't even validly
      get off the ground), as `\Z` and `\X` already do.
    
    * src/roff/groff/tests/some_escapes_accept_newline_delimiters.sh:
      Delete.
    * src/roff/groff/groff.am (groff_TESTS): Drop test.
    
    * doc/groff.texi.in (Delimiters):
    * man/groff.7.man (Delimiters):
    * NEWS: De-document support for newlines as delimiters.
    
    Fixes <https://savannah.gnu.org/bugs/?63142>.

 ChangeLog                                          |  30 +++++
 NEWS                                               |   3 +
 doc/groff.texi.in                                  |  26 -----
 man/groff.7.man                                    |  17 ---
 src/roff/groff/groff.am                            |   1 -
 .../some_escapes_accept_newline_delimiters.sh      | 124
---------------------
 src/roff/troff/input.cpp                           |  34 +++++-
 7 files changed, 61 insertions(+), 174 deletions(-)
 delete mode 100755
src/roff/groff/tests/some_escapes_accept_newline_delimiters.sh


I'm starting to think I got myself into trouble with you with the "as a bonus"
part.  ;-)


    _______________________________________________________

Reply to this item at:

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

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

Attachment: signature.asc
Description: PGP signature

Reply via email to