gbranden pushed a commit to branch master
in repository groff.

commit 8a0b4c77fb1f53fcfc837db291188cd7c8b4a766
Author: G. Branden Robinson <[email protected]>
AuthorDate: Sat Apr 18 10:12:54 2026 -0500

    src/roff/troff/env.cpp: Fix code style nit.
    
    * src/roff/troff/env.cpp (select_font): Parenthesize formally complex
      expression.
---
 ChangeLog              | 5 +++++
 src/roff/troff/env.cpp | 4 +++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index c34a276a0..a3e3e6ad1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2026-04-18  G. Branden Robinson <[email protected]>
+
+       * src/roff/troff/env.cpp (select_font): Fix code style nit.
+       Parenthesize formally complex expression.
+
 2026-04-17  G. Branden Robinson <[email protected]>
 
        [troff]: Trivially refactor.
diff --git a/src/roff/troff/env.cpp b/src/roff/troff/env.cpp
index 9121bb98c..d96e2908d 100644
--- a/src/roff/troff/env.cpp
+++ b/src/roff/troff/env.cpp
@@ -1341,9 +1341,11 @@ void select_font(symbol s)
   else {
     const char *p = s.contents();
     assert(*p != 0 /* nullptr */);
+    // Silently ignore a leading minus sign so we can issue a range
+    // warning later.
     if ((csdigit(*p)) || ('-' == *p))
       p++;
-    for (; p != 0 /* nullptr */ && *p != '\0'; p++)
+    for (; (p != 0 /* nullptr */) && (*p != '\0'); p++)
       if (!csdigit(*p)) {
        is_number = false;
        break;

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

Reply via email to