[bug #64628] [pic] generated `lf` requests no longer set file name

[G. Branden Robinson]

Update of bug #64628 (project groff):

                  Status:             In Progress => Fixed                  
             Open/Closed:                    Open => Closed                 
         Planned Release:                    None => 1.24.0                 

    _______________________________________________________

Follow-up Comment #7:


commit 76af4781348027740537e4f3e620af55f4e40efa
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
AuthorDate: Mon Sep 25 05:42:19 2023 -0500

    [pic]: Fix Savannah #64628 (incorrect `lf` usage).

    * src/preproc/pic/troff.cpp (strsame): Add utility function for
      comparing two C strings for identical content, handling null pointer
      arguments as `strcmp()` does not.

      (troff_output::set_location): Refactor; write the two-argument form of
      the `lf` request if the current and last seen file names _don't_
      match.  Problem introduced by me in commit 705be31107, 29 July.  Also
      fix heap memory leak when repeatedly updating
      `troff_output::last_filename` member variable.

    Fixes <https://savannah.gnu.org/bugs/?64628>.  Thanks to Bjarni Ingi
    Gislason for the report.




    _______________________________________________________

Reply to this item at:

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

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