From: Eli Zaretskii <eliz@gnu.org>
To: Andrei Pikas <gdb@mail.api.win>
Cc: gdb-patches@sourceware.org, gdb@mail.api.win
Subject: Re: [PATCH v6] Add an option with a color type.
Date: Sun, 16 Oct 2022 19:45:46 +0300 [thread overview]
Message-ID: <83czar4szp.fsf@gnu.org> (raw)
In-Reply-To: <20221016162809.32801-1-gdb@mail.api.win> (message from Andrei Pikas on Sun, 16 Oct 2022 19:28:09 +0300)
> From: Andrei Pikas <gdb@mail.api.win>
> Cc: eliz@gnu.org,
> Andrei Pikas <gdb@mail.api.win>
> Date: Sun, 16 Oct 2022 19:28:09 +0300
>
> Colors can be specified as "none" for terminal's default color, as a name of
> one of the eight standard colors of ISO/IEC 6429 "black", "red", "green", etc.,
> as an RGB hexadecimal tripplet #RRGGBB for 24-bit TrueColor, or as an
> integer from 0 to 255. Integers 0 to 7 are the synonyms for the standard
> colors. Integers 8-15 are used for the so-called bright colors from the
> aixterm extended 16-color palette. Integers 16-255 are the indexes into xterm
> extended 256-color palette (usually 6x6x6 cube plus gray ramp). In
> general, 256-color palette is terminal dependent and sometimes can be
> changed with OSC 4 sequences, e.g. "\033]4;1;rgb:00/FF/00\033\\".
>
> It is the responsibility of the user to verify that the terminal supports
> the specified colors.
>
> PATCH v5 changes: documentation fixed.
> PATCH v6 changes: documentation fixed.
> ---
> gdb/Makefile.in | 2 +
> gdb/NEWS | 29 ++
> gdb/cli/cli-cmds.c | 7 +
> gdb/cli/cli-decode.c | 174 +++++++++
> gdb/cli/cli-decode.h | 21 +
> gdb/cli/cli-option.c | 44 +++
> gdb/cli/cli-option.h | 21 +
> gdb/cli/cli-setshow.c | 21 +
> gdb/cli/cli-style.c | 49 +--
> gdb/cli/cli-style.h | 4 +-
> gdb/command.h | 26 +-
> gdb/doc/gdb.texinfo | 38 +-
> gdb/doc/guile.texi | 104 +++++
> gdb/doc/python.texi | 97 +++++
> gdb/guile/guile-internal.h | 9 +
> gdb/guile/guile.c | 1 +
> gdb/guile/scm-color.c | 445 ++++++++++++++++++++++
> gdb/guile/scm-param.c | 29 +-
> gdb/python/py-color.c | 346 +++++++++++++++++
> gdb/python/py-color.h | 35 ++
> gdb/python/py-param.c | 34 +-
> gdb/python/python-internal.h | 2 +
> gdb/python/python.c | 8 +
> gdb/testsuite/gdb.base/style.exp | 209 +++++++++-
> gdb/testsuite/gdb.guile/scm-color.exp | 114 ++++++
> gdb/testsuite/gdb.guile/scm-parameter.exp | 43 +++
> gdb/testsuite/gdb.python/py-color.exp | 101 +++++
> gdb/testsuite/gdb.python/py-parameter.exp | 53 +++
> gdb/testsuite/lib/gdb-utils.exp | 22 +-
> gdb/top.c | 13 +
> gdb/ui-style.c | 331 ++++++++++++----
> gdb/ui-style.h | 142 ++++++-
> gdb/unittests/style-selftests.c | 6 +-
> 33 files changed, 2420 insertions(+), 160 deletions(-)
> create mode 100644 gdb/guile/scm-color.c
> create mode 100644 gdb/python/py-color.c
> create mode 100644 gdb/python/py-color.h
> create mode 100644 gdb/testsuite/gdb.guile/scm-color.exp
> create mode 100644 gdb/testsuite/gdb.python/py-color.exp
OK for the documentation parts, thanks.
next prev parent reply other threads:[~2022-10-16 16:46 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-16 11:40 [PATCH v4] " Andrei Pikas
2022-10-16 13:45 ` Eli Zaretskii
2022-10-16 16:03 ` [PATCH v5] " Andrei Pikas
2022-10-16 16:22 ` Eli Zaretskii
2022-10-16 16:28 ` [PATCH v6] " Andrei Pikas
2022-10-16 16:45 ` Eli Zaretskii [this message]
2024-04-19 19:33 ` Tom Tromey
2024-04-19 19:52 ` Andrei Pikas
2024-04-19 20:19 ` Tom Tromey
2024-04-20 18:24 ` Tom Tromey
2024-04-20 18:32 ` Andrei Pikas
2024-05-11 15:17 ` Andrei Pikas
2024-05-13 19:02 ` Tom Tromey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83czar4szp.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=gdb@mail.api.win \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).