gbranden pushed a commit to branch master
in repository groff.

commit b6e64ebbacac55526da60e6a580aa05838c892cd
Author: G. Branden Robinson <[email protected]>
AuthorDate: Thu May 15 15:51:03 2025 -0500

    src/roff/troff/input.cpp: Fix code style nit.
    
    * src/roff/troff/input.cpp (token::next, exit_troff): Assign Boolean,
      not integer, literal, to variable of type `bool`.
---
 ChangeLog                | 7 +++++++
 src/roff/troff/input.cpp | 8 ++++----
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 3dfea0a80..5360dfb1b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2025-05-15  G. Branden Robinson <[email protected]>
+
+       src/roff/troff/input.cpp: Fix code style nit.
+
+       * src/roff/troff/input.cpp (token::next, exit_troff): Assign
+       Boolean, not integer, literal, to variable of type `bool`.
+
 2025-05-15  G. Branden Robinson <[email protected]>
 
        * src/roff/troff/node.cpp (class real_output_file): Demote
diff --git a/src/roff/troff/input.cpp b/src/roff/troff/input.cpp
index d2eff7c93..2c5a10edf 100644
--- a/src/roff/troff/input.cpp
+++ b/src/roff/troff/input.cpp
@@ -2049,7 +2049,7 @@ void token::next()
        type = TOKEN_END_TRAP;
        return;
       case LAST_PAGE_EJECTOR:
-       seen_last_page_ejector = 1;
+       seen_last_page_ejector = true;
        // fall through
       case PAGE_EJECTOR:
        type = TOKEN_PAGE_EJECTOR;
@@ -2937,10 +2937,10 @@ void exit_troff()
     // TODO: Resolve the follwing case and enable the assertion.
     // $ printf '.DS\n.DE\n' | ./build/test-groff -ms
     // troff: ../src/roff/troff/input.cpp:2937: void exit_troff():
-    //   Assertion `seen_last_page_ejector == 1' failed.
+    //   Assertion `seen_last_page_ejector' failed.
     // .../build/groff: error: troff: Aborted (core dumped)
-    //assert(seen_last_page_ejector == 1);
-    seen_last_page_ejector = 1;        // should be set already
+    //assert(seen_last_page_ejector);
+    seen_last_page_ejector = true;     // should be set already
     topdiv->set_ejecting();
     push_page_ejector();
     topdiv->space(topdiv->get_page_length(), true /* forcing */);

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

Reply via email to