gbranden pushed a commit to branch master
in repository groff.

commit d2b8bc0cc9227e8410cd03f01db24933dbaa4347
Author: G. Branden Robinson <[email protected]>
AuthorDate: Thu Apr 3 20:02:56 2025 -0500

    [troff]: Fix JSON syntax goof.
    
    * src/roff/troff/input.cpp (macro::json_dump): Always print a comma
      before dumping contents, since the "length" datum always precedes it.
      Fixes malformed output of `.pm .T`.
---
 ChangeLog                | 6 ++++++
 src/roff/troff/input.cpp | 3 +--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 68b053ca4..4b11fc8c0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2025-04-03  G. Branden Robinson <[email protected]>
+
+       * src/roff/troff/input.cpp (macro::json_dump): Always print a
+       comma before dumping contents, since the "length" datum always
+       precedes it.  Fixes malformed output of `.pm .T`.
+
 2025-04-03  G. Branden Robinson <[email protected]>
 
        * src/roff/troff/input.cpp (print_macros): Fix infloop when
diff --git a/src/roff/troff/input.cpp b/src/roff/troff/input.cpp
index 5de7c7859..d4e951407 100644
--- a/src/roff/troff/input.cpp
+++ b/src/roff/troff/input.cpp
@@ -3724,8 +3724,7 @@ void macro::json_dump()
     errprint(", ");
   errprint("\"length\": %1", len);
   if (p != 0 /* nullptr */) {
-    if (need_comma)
-      errprint(", ");
+    errprint(", ");
     if (is_a_diversion)
       p->json_dump_diversion();
     else

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

Reply via email to