[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-readline] [PATCH] Enable visibility annotations
From: |
Chet Ramey |
Subject: |
Re: [Bug-readline] [PATCH] Enable visibility annotations |
Date: |
Thu, 14 Apr 2016 10:59:25 -0400 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 |
On 4/14/16 6:47 AM, Pedro Alves wrote:
> FYI, this is probably doing to break (at least) gdb against system
> readline. gdb relies on accessing a few private readline symbols... :-/
>
> E.g.:
>
> tui/tui-io.c:437: extern int _rl_echoing_p;
> completer.c:1677: extern int _rl_complete_mark_directories;
> completer.c:1770:extern int _rl_completion_prefix_display_length;
> completer.c:1771:extern int _rl_print_completions_horizontally;
>
> Unless/until someone fixes these and adds the missing public APIs
> to readline,
These aren't exactly the result of `missing public APIs'. echoing_p is
a saved value indicating whether or not the terminal flags readline
inherits include ECHO -- something an application can easily check for
itself.
The others are values for settable variables and should be retrieved
using rl_variable_value (variable_name). Since it returns a string,
you can use the public macro RL_BOOLEAN_VARIABLE_VALUE to turn it into
an int.
Bash doesn't use any _rl_ readline symbols, so I know it can be done.
--
``The lyf so short, the craft so long to lerne.'' - Chaucer
``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, ITS, CWRU address@hidden http://cnswww.cns.cwru.edu/~chet/
- Re: [Bug-readline] [PATCH] Enable visibility annotations, (continued)
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Chet Ramey, 2016/04/22
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Pedro Alves, 2016/04/22
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Yury Gribov, 2016/04/22
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Pedro Alves, 2016/04/22
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Yury Gribov, 2016/04/22
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Pedro Alves, 2016/04/22
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Mike Frysinger, 2016/04/14
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Chet Ramey, 2016/04/14
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Mike Frysinger, 2016/04/14
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Mike Frysinger, 2016/04/14
- Re: [Bug-readline] [PATCH] Enable visibility annotations,
Chet Ramey <=
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Pedro Alves, 2016/04/14
Re: [Bug-readline] [PATCH] Enable visibility annotations, Doug Evans, 2016/04/18
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Yury Gribov, 2016/04/20
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Pedro Alves, 2016/04/20
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Yury Gribov, 2016/04/20
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Pedro Alves, 2016/04/20
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Yury Gribov, 2016/04/20
- Re: [Bug-readline] [PATCH] Enable visibility annotations, Chet Ramey, 2016/04/20
Re: [Bug-readline] [PATCH] Enable visibility annotations, Chet Ramey, 2016/04/20