Bug 30411 - [gdb/cli] CLI in ansi terminal has off-by-one width problem (info sources output)
Summary: [gdb/cli] CLI in ansi terminal has off-by-one width problem (info sources out...
Status: RESOLVED FIXED
Alias: None
Product: gdb
Classification: Unclassified
Component: cli (show other bugs)
Version: HEAD
: P2 normal
Target Milestone: 14.1
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-02 09:20 UTC by Tom de Vries
Modified: 2023-05-12 09:46 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom de Vries 2023-05-02 09:20:49 UTC
Consider a terminal with 70 columns.

With TERM=xterm we have for info sources:
...
/data/vries/hello.c, 
/usr/lib64/gcc/x86_64-suse-linux/7/include/stddef.h, 
/usr/include/bits/types.h, /usr/include/bits/types/struct_FILE.h, 
/usr/include/bits/types/FILE.h, /usr/include/stdio.h, 
/usr/include/bits/sys_errlist.h, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/../sysdeps/x86_64/start.S, /home/abuild/rpmbuild/BUILD/glibc-2.31/csu/init.c, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/../sysdeps/x86_64/crti.S, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/elf-init.c, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/../sysdeps/x86_64/crtn.S
...

With TERM=ansi, we have instead:
...
/data/vries/hello.c, 
/usr/lib64/gcc/x86_64-suse-linux/7/include/stddef.h, 
/usr/include/bits/types.h, /usr/include/bits/types/struct_FILE.h, 
/usr/include/bits/types/FILE.h, /usr/include/stdio.h, 
/usr/include/bits/sys_errlist.h, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/../sysdeps/x86_64/start.S, /home/abuild/rpmbuild/BUILD/glibc-2.31/csu/init.c, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/../sysdeps/x86_64/crti.S, /
home/abuild/rpmbuild/BUILD/glibc-2.31/csu/elf-init.c, 
/home/abuild/rpmbuild/BUILD/glibc-2.31/csu/../sysdeps/x86_64/crtn.S
...

Note the line break after "/".
Comment 1 Sourceware Commits 2023-05-05 10:20:20 UTC
The master branch has been updated by Tom de Vries <vries@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c2a0fca06af371adc360c2da6bafc23651e5761e

commit c2a0fca06af371adc360c2da6bafc23651e5761e
Author: Tom de Vries <tdevries@suse.de>
Date:   Fri May 5 12:20:20 2023 +0200

    [gdb/testsuite] Add gdb.tui/wrap-line.exp
    
    Add a test-case that tests prompt edit wrapping behaviour in the tuiterm, both
    for CLI and TUI, both with auto-detected and hard-coded width.
    
    In the CLI case with auto-detected width we run into PR cli/30411, so add a
    KFAIL for that failure mode.
    
    Tested on x86_64-linux.
Comment 2 Sourceware Commits 2023-05-12 09:43:48 UTC
The master branch has been updated by Tom de Vries <vries@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f0f6df0a04fe521ff4df9b74981a624fa2583e3a

commit f0f6df0a04fe521ff4df9b74981a624fa2583e3a
Author: Tom de Vries <tdevries@suse.de>
Date:   Fri May 12 11:43:41 2023 +0200

    [gdb/cli] Fix wrapping for TERM=ansi
    
    I. Auto-detected width (xterm vs. ansi)
    
    Say we have a terminal with a width of 40 chars:
    ...
    $ echo $COLUMNS
    40
    ...
    
    With TERM=xterm, we report a width of 40 chars:
    ...
    $ TERM=xterm gdb
    (gdb) show width
    Number of characters gdb thinks are in a line is 40.
    ...
    
    And with TERM=ansi, a width of 39 chars:
    ...
    $ TERM=ansi gdb
    (gdb) show width
    Number of characters gdb thinks are in a line is 39.
    ...
    
    Gdb uses readline to auto-detect screen size, and readline decides in the
    TERM=ansi case that the terminal does not have reliable auto-wrap, and
    consequently decides to hide the last terminal column from the readline user
    (in other words GDB), hence we get 39 instead of 40.
    
    II. Types of wrapping
    
    Looking a bit more in detail inside gdb, it seems there are two types of
    wrapping:
    - readline wrapping (in other words, prompt edit wrapping), and
    - gdb output wrapping (can be observed by issuing "info sources").
      This type of wrapping attempts to wrap some of the gdb output earlier
      than the indicated width, to not break lines in inconvenient places.
    
    III. Readline wrapping, auto-detected screen size
    
    Let's investigate readline wrapping with the auto-detected screen widths.
    
    First, let's try with xterm:
    ...
    $ TERM=xterm gdb
    (gdb) 7890123456789012345678901234567890
    123
    ...
    That looks as expected, wrapping occurs after 40 chars.
    
    Now, let's try with ansi:
    ...
    $ TERM=ansi gdb
    (gdb) 78901234567890123456789012345678
    90123
    ...
    It looks like wrapping occurred after 38, while readline should be capable of
    wrapping after 39 chars.
    
    This is caused by readline hiding the last column, twice.
    
    In more detail:
    - readline detects the screen width: 40,
    - readline hides the last column, setting the readline screen width to 39,
    - readline reports 39 to gdb as screen width,
    - gdb sets its width setting to 39,
    - gdb sets readline screen width to 39,
    - readline hides the last column, again, setting the readline screen width to
      38.
    
    This is reported as PR cli/30346.
    
    IV. gdb output wrapping, auto-detected screen size
    
    Say we set the terminal width to 56. With TERM=xterm, we have:
    ...
    /home/abuild/rpmbuild/BUILD/glibc-2.31/csu/elf-init.c,
    /data/vries/hello.c,
    ...
    but with TERM=ansi:
    ...
    /home/abuild/rpmbuild/BUILD/glibc-2.31/csu/elf-init.c, /
    data/vries/hello.c,
    ...
    
    So what happened here?  With TERM=ansi, the width setting is auto-detected to
    55, and gdb assumes the terminal inserts a line break there, which it doesn't
    because the terminal width is 56.
    
    This is reported as PR cli/30411.
    
    V. Fix PRs
    
    Fix both mentioned PRs by taking into account the hidden column when readline
    reports the screen width in init_page_info, and updating chars_per_line
    accordingly.
    
    Note that now we report the same width for both TERM=xterm and TERM=ansi,
    which is much clearer.
    
    The point where readline respectively expects or ensures wrapping is still
    indicated by "maint info screen", for xterm:
    ...
    Number of characters readline reports are in a line is 40.
    ...
    and ansi:
    ...
    Number of characters readline reports are in a line is 39.
    ...
    
    VI. Testing
    
    PR cli/30346 is covered by existing regression tests gdb.base/wrap-line.exp
    and gdb.tui/wrap-line.exp, so remove the KFAILs there.
    
    I didn't manage to come up with a regression test for PR cli/30411.  Perhaps
    that would be easier if we had a maintenance command that echoes its arguments
    while applying gdb output wrapping.
    
    Tested on x86_64-linux.
    
    PR cli/30346
    PR cli/30411
    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30346
    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30411
Comment 3 Tom de Vries 2023-05-12 09:46:16 UTC
Fixed.