[gdb/tui] Refactor prefresh call in tui_source_window_base::refresh_window
authorTom de Vries <tdevries@suse.de>
Mon, 13 Nov 2023 20:22:50 +0000 (21:22 +0100)
committerTom de Vries <tdevries@suse.de>
Mon, 13 Nov 2023 20:22:50 +0000 (21:22 +0100)
commit70911fd87f261d3726fea4699a4ffdd5623482b7
tree6b06748001671adbc5cd5c768c2f95d935900452
parent284c40cb7dc72756e60b8754f719ba56370350a4
[gdb/tui] Refactor prefresh call in tui_source_window_base::refresh_window

Currently the call to prefresh in tui_source_window_base::refresh_window looks
like:
...
  prefresh (m_pad.get (), 0, pad_x, y + 1, x + left_margin,
    y + m_content.size (), x + left_margin + view_width - 1);
...

This is hard to parse.  It's not obvious what the arguments mean, and there's
repetition in the argument calculation.

Fix this by rewriting the call as follows:
- use sminrow, smincol, smaxrow and smaxcol variables for the last
  4 arguments, and
- calculate the smaxrow and smaxcol variables based on the sminrow and
  smincol variables.

Tested on x86_64-linux.

Approved-By: Tom Tromey <tom@tromey.com>
gdb/tui/tui-winsource.c