gbranden pushed a commit to branch master
in repository groff.

commit fdf5255268a709d88aba94fe20bc118617d16873
Author: G. Branden Robinson <[email protected]>
AuthorDate: Sun Sep 8 06:41:50 2024 -0500

    [troff]: Drop unused logic.
    
    * src/roff/troff/node.cpp (troff_output_file::start_device_extension):
      Stop calling `do_motion()` in argumentless version, which is called
      only when writing HTML (specifically, when inlining images).  It
      seemed to accomplish nothing.  No tests fail and "pic.html", a
      document with many images, continues to render fine.  In
      device-independent output, a pair of absolute positioning commands
      ('H' and 'V') moves to a redundant place, but the HTML produces is
      identical (except for a date stamp in a comment).
---
 ChangeLog               | 12 ++++++++++++
 src/roff/troff/node.cpp |  1 -
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 51b071809..e37021d04 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2024-09-08  G. Branden Robinson <[email protected]>
+
+       * src/roff/troff/node.cpp
+       (troff_output_file::start_device_extension): Stop calling
+       `do_motion()` in argumentless version, which is called only when
+       writing HTML (specifically, when inlining images).  It seemed to
+       accomplish nothing.  No tests fail and "pic.html", a document
+       with many images, continues to render fine.  In
+       device-independent output, a pair of absolute positioning
+       commands ('H' and 'V') moves to a redundant place, but the HTML
+       produces is identical (except for a date stamp in a comment).
+
 2024-09-08  G. Branden Robinson <[email protected]>
 
        [troff]: Trivially refactor.
diff --git a/src/roff/troff/node.cpp b/src/roff/troff/node.cpp
index 0a7b8bffe..4082a3eef 100644
--- a/src/roff/troff/node.cpp
+++ b/src/roff/troff/node.cpp
@@ -902,7 +902,6 @@ void troff_output_file::start_device_extension(tfont *tf, 
color *gcol,
 void troff_output_file::start_device_extension()
 {
   flush_tbuf();
-  do_motion();
   put("x X ");
 }
 

_______________________________________________
Groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit

Reply via email to