gbranden pushed a commit to branch master
in repository groff.

commit ebc25e82e09b0d4881ca1f362855baee2eaf99b6
Author: G. Branden Robinson <[email protected]>
AuthorDate: Wed Mar 25 11:53:03 2026 -0500

    [libgroff]: Boolify and rename variable.
    
    ...a static class member variable, specifically.
    
    * src/include/cmap.h (class cmap_init): `initialised` of type `int`
      becomes `is_initialised` of type `bool`.
    
    * src/libs/libgroff/cmap.cpp: Assign to it with Boolean, not integer,
      literals.
---
 ChangeLog                  | 9 +++++++++
 src/include/cmap.h         | 2 +-
 src/libs/libgroff/cmap.cpp | 6 +++---
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index aaec72d87..f655825cf 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -19,6 +19,15 @@
        * src/devices/grops/ps.cpp: Drop unnecessary preprocessor
        inclusions of "paper.h" header file.
 
+2026-03-25  G. Branden Robinson <[email protected]>
+
+       [libgroff]: Boolify and rename static class member variable.
+
+       * src/include/cmap.h (class cmap_init): `initialised` of type
+       `int` becomes `is_initialised` of type `bool`.
+       * src/libs/libgroff/cmap.cpp: Assign to it with Boolean, not
+       integer, literals.
+
 2026-03-25  G. Branden Robinson <[email protected]>
 
        * src/include/error.h: Declare `fatal_error_exit()` here...
diff --git a/src/include/cmap.h b/src/include/cmap.h
index 2edbaf914..bae445373 100644
--- a/src/include/cmap.h
+++ b/src/include/cmap.h
@@ -48,7 +48,7 @@ extern cmap cmlower;
 extern cmap cmupper;
 
 static class cmap_init {
-  static int initialised;
+  static bool is_initialised;
 public:
   cmap_init();
 } _cmap_init;
diff --git a/src/libs/libgroff/cmap.cpp b/src/libs/libgroff/cmap.cpp
index 8e60e7732..092511cf6 100644
--- a/src/libs/libgroff/cmap.cpp
+++ b/src/libs/libgroff/cmap.cpp
@@ -44,17 +44,17 @@ cmap::cmap(cmap_builtin)
   // these are initialised by cmap_init::cmap_init()
 }
 
-int cmap_init::initialised = 0;
+bool cmap_init::is_initialised = false;
 
 cmap_init::cmap_init()
 {
-  if (initialised)
+  if (is_initialised)
     return;
-  initialised = 1;
   for (int i = 0; i <= UCHAR_MAX; i++) {
     cmupper.v[i] = ISASCII(i) && islower(i) ? toupper(i) : i;
     cmlower.v[i] = ISASCII(i) && isupper(i) ? tolower(i) : i;
   }
+  is_initialised = true;
 }
 
 // Local Variables:

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

Reply via email to