This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] TUI: Expand TABs into spaces
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Doug Evans <xdje42 at gmail dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Fri, 16 Jan 2015 18:43:58 +0200
- Subject: Re: [PATCH] TUI: Expand TABs into spaces
- Authentication-results: sourceware.org; auth=none
- References: <83k3149k5b dot fsf at gnu dot org> <837fwn2cvc dot fsf at gnu dot org> <CAP9bCMSbzc3q5CmJQDZyL0utP_x6Sp8ekHaTHzJHyVCJRwwcWA at mail dot gmail dot com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Fri, 16 Jan 2015 08:32:15 -0800
> From: Doug Evans <xdje42@gmail.com>
> Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
>
> >> 2015-01-03 Eli Zaretskii <eliz@gnu.org>
> >>
> >> * tui/tui-regs.c (tui_register_format): Expand TABs into the
> >> appropriate number of spaces.
> >>
> >> * tui/tui-io.c (tui_puts, tui_redisplay_readline): Expand TABs
> >> into the appropriate number of spaces.
>
> I'd have to read the patch more to say it's ok,
> but one thing that is missing are comments
> explaining *why* we are expanding tabs into spaces.
You mean, the fact that the Windows port of ncurses doesn't? Sure,
can do that. But note that a similar feature in displaying the source
doesn't have any such comments.