[PATCH 2/2] gdb/testsuite: rename _cur_x/_cur_y to _cur_col/_cur_row in lib/tuiterm.exp

Tom Tromey tom@tromey.com
Mon Jan 11 20:56:31 GMT 2021


>>>>> "Simon" == Simon Marchi via Gdb-patches <gdb-patches@sourceware.org> writes:

Simon> I am having trouble remembering which of _cur_x/_cur_y is columns and
Simon> which is rows, so renaming them helps.  We already have _rows and _cols
Simon> to represent the terminal size, so I think that makes sense to name the
Simon> "_cur" variables accordingly.

Hah, i have the opposite problem, where "x" and "y" are clear but I
have to think about row/column.

Simon> gdb/testsuite/ChangeLog:

Simon> 	* lib/tuiterm.exp: Rename _cur_x/_cur_y to _cur_col/_cur_row.

Looks good.  Thanks.

Tom


More information about the Gdb-patches mailing list