gbranden pushed a commit to branch master in repository groff. commit be95332b1ec2f204ec0b8b12c625b4df91dcb33d Author: G. Branden Robinson <g.branden.robin...@gmail.com> AuthorDate: Fri Feb 21 02:47:26 2025 -0600
[troff]: Trivially refactor (2/2). Rename symbols for clarity. * src/roff/troff/input.cpp (get_charinfo_by_number): Rename function in declaration and definition from this... (get_charinfo_by_index): ...to this. (token::get_char) (token::add_to_zero_width_node_list, token::process) (name_to_glyph): Update call sites. * src/roff/troff/input.cpp: Rename global object of `dictionary` type from `numbered_charinfo_dictionary` to `indexed_charinfo_dictionary`. * src/roff/troff/input.cpp (get_charinfo_by_index): Rename static local array of `charinfo` type from `number_table` to `index_table`. --- ChangeLog | 17 +++++++++++++++++ src/roff/troff/input.cpp | 26 +++++++++++++------------- 2 files changed, 30 insertions(+), 13 deletions(-) diff --git a/ChangeLog b/ChangeLog index 0a0f85faf..0a66b1632 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,20 @@ +2025-02-21 G. Branden Robinson <g.branden.robin...@gmail.com> + + * src/roff/troff/input.cpp (get_charinfo_by_number): Rename + function in declaration and definition from this... + (get_charinfo_by_index): ...to this. + (token::get_char) + (token::add_to_zero_width_node_list, token::process) + (name_to_glyph): Update call sites. + + * src/roff/troff/input.cpp: Rename global object of `dictionary` + type from `numbered_charinfo_dictionary` to + `indexed_charinfo_dictionary`. + + * src/roff/troff/input.cpp (get_charinfo_by_index): Rename + static local array of `charinfo` type from `number_table` to + `index_table`. + 2025-02-21 G. Branden Robinson <g.branden.robin...@gmail.com> [troff]: Unpublish function that doesn't need to be visible diff --git a/src/roff/troff/input.cpp b/src/roff/troff/input.cpp index d30862a36..54072a520 100644 --- a/src/roff/troff/input.cpp +++ b/src/roff/troff/input.cpp @@ -8148,7 +8148,7 @@ void define_class() skip_line(); } -static charinfo *get_charinfo_by_number(int n); // forward declaration +static charinfo *get_charinfo_by_index(int n); // forward declaration charinfo *token::get_char(bool required) { @@ -8157,7 +8157,7 @@ charinfo *token::get_char(bool required) if (type == TOKEN_SPECIAL_CHAR) return get_charinfo(nm); if (type == TOKEN_INDEXED_CHAR) - return get_charinfo_by_number(val); + return get_charinfo_by_index(val); if (type == TOKEN_ESCAPE) { if (escape_char != 0) return charset_table[escape_char]; @@ -8235,7 +8235,7 @@ bool token::add_to_zero_width_node_list(node **pp) nd = 0 /* nullptr */; break; case TOKEN_INDEXED_CHAR: - *pp = (*pp)->add_char(get_charinfo_by_number(val), curenv, &w, &s); + *pp = (*pp)->add_char(get_charinfo_by_index(val), curenv, &w, &s); break; case TOKEN_RIGHT_BRACE: break; @@ -8328,7 +8328,7 @@ void token::process() nd = 0; break; case TOKEN_INDEXED_CHAR: - curenv->add_char(get_charinfo_by_number(val)); + curenv->add_char(get_charinfo_by_index(val)); break; case TOKEN_REQUEST: // handled in process_input_stack() @@ -10206,31 +10206,31 @@ void charinfo::dump() symbol UNNAMED_SYMBOL("---"); -// For numbered characters not between 0 and 255, we make a symbol out +// For indexed characters not between 0 and 255, we make a symbol out // of the number and store them in this dictionary. -dictionary numbered_charinfo_dictionary(11); +dictionary indexed_charinfo_dictionary(11); -static charinfo *get_charinfo_by_number(int n) +static charinfo *get_charinfo_by_index(int n) { - static charinfo *number_table[256]; + static charinfo *index_table[256]; if (n >= 0 && n < 256) { - charinfo *ci = number_table[n]; + charinfo *ci = index_table[n]; if (0 /*nullptr */ == ci) { ci = new charinfo(UNNAMED_SYMBOL); ci->set_number(n); - number_table[n] = ci; + index_table[n] = ci; } return ci; } else { symbol ns(i_to_a(n)); - charinfo *ci = (charinfo *)numbered_charinfo_dictionary.lookup(ns); + charinfo *ci = (charinfo *)indexed_charinfo_dictionary.lookup(ns); if (0 /*nullptr */ == ci) { ci = new charinfo(UNNAMED_SYMBOL); ci->set_number(n); - (void) numbered_charinfo_dictionary.lookup(ns, ci); + (void) indexed_charinfo_dictionary.lookup(ns, ci); } return ci; } @@ -10256,7 +10256,7 @@ glyph *name_to_glyph(const char *nm) glyph *number_to_glyph(int n) { - return get_charinfo_by_number(n)->as_glyph(); + return get_charinfo_by_index(n)->as_glyph(); } const char *glyph_to_name(glyph *g) _______________________________________________ groff-commit mailing list groff-commit@gnu.org https://lists.gnu.org/mailman/listinfo/groff-commit