gbranden pushed a commit to branch master
in repository groff.
commit e074c55f2794ff7e8c49a3a103383dd3fde73849
Author: G. Branden Robinson <[email protected]>
AuthorDate: Wed Nov 26 00:18:49 2025 -0600
[troff]: Fix code style nit.
* src/roff/troff/input.cpp (do_get_long_name, read_size):
* src/roff/troff/reg.cpp (assign_register_format_request): Parenthesize
formally complex expressions.
---
ChangeLog | 6 ++++++
src/roff/troff/input.cpp | 8 ++++----
src/roff/troff/reg.cpp | 5 ++++-
3 files changed, 14 insertions(+), 5 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index b96e25100..03bbda100 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2025-11-26 G. Branden Robinson <[email protected]>
+
+ * src/roff/troff/input.cpp (do_get_long_name, read_size):
+ * src/roff/troff/reg.cpp (assign_register_format_request): Fix
+ code style nit; parenthesize formally complex expressions.
+
2025-11-25 G. Branden Robinson <[email protected]>
* src/roff/troff/input.cpp (do_name_test, return_macro_request):
diff --git a/src/roff/troff/input.cpp b/src/roff/troff/input.cpp
index d4652c329..ef1db1d60 100644
--- a/src/roff/troff/input.cpp
+++ b/src/roff/troff/input.cpp
@@ -3158,7 +3158,7 @@ static symbol do_get_long_name(bool required, char
end_char)
buf_size = new_buf_size;
delete[] old_buf;
}
- if ((buf[i] = tok.ch()) == 0U || (buf[i] == end_char))
+ if (((buf[i] = tok.ch()) == 0U) || (buf[i] == end_char))
break;
i++;
tok.next();
@@ -6097,13 +6097,13 @@ static bool read_size(int *x) // \s
token start(tok);
tok.next();
c = tok.ch();
- if (!inc && (c == '-' || c == '+')) {
- inc = c == '+' ? 1 : -1;
+ if (!inc && ((c == '-') || (c == '+'))) {
+ inc = (c == '+') ? 1 : -1;
tok.next();
}
if (!read_measurement(&val, 'z'))
return false;
- if (!(start.ch() == '[' && tok.ch() == ']') && start != tok) {
+ if (!((start.ch() == '[') && (tok.ch() == ']')) && (start != tok)) {
if (start.ch() == '[')
error("missing ']' in type size escape sequence");
else
diff --git a/src/roff/troff/reg.cpp b/src/roff/troff/reg.cpp
index 1314ba23e..c1ae3a371 100644
--- a/src/roff/troff/reg.cpp
+++ b/src/roff/troff/reg.cpp
@@ -456,7 +456,10 @@ static void assign_register_format_request()
} while (csdigit(tok.ch()));
r->alter_format('1', n);
}
- else if (c == 'i' || c == 'I' || c == 'a' || c == 'A')
+ else if ((c == 'i')
+ || (c == 'I')
+ || (c == 'a')
+ || (c == 'A'))
r->alter_format(c);
else if (!has_arg())
warning(WARN_MISSING, "register interpolation format assignment"
_______________________________________________
groff-commit mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/groff-commit