Re: [PATCH 16/16] Document the "set style" commands

Eli Zaretskii
2018-11-28T06:51:55Z

> From: Tom Tromey <tom@tromey.com>
> Cc: Tom Tromey <tom@tromey.com>
> Date: Tue, 27 Nov 2018 17:14:35 -0700
>
> +For example, the style of file names can be controlled using the
> +@code{set style filename} group of commands:
> +
> +@table @code
> +@kindex set style filename background
> +@item set style filename background @var{color}
> +Set the background to @var{color}. Valid colors are @samp{none}
> +(meaning the terminal's default color), @samp{black}, @samp{red},
> +@samp{green}, @samp{yellow}, @samp{vlue}, @samp{magenta}, @samp{cyan},
> +and@samp{white}.
> +
> +@kindex set style filename foreground
> +@item set style filename foreground @var{color}
> +Set the foreground to @var{color}. Valid colors are @samp{none}
> +(meaning the terminal's default color), @samp{black}, @samp{red},
> +@samp{green}, @samp{yellow}, @samp{vlue}, @samp{magenta}, @samp{cyan},
> +and@samp{white}.
> +
> +@kindex set style filename intensity
> +@item set style filename intensity @var{value}
> +Set the intensity to @var{value}. Valid intensities are @samp{normal}
> +(the default), @samp{bold}, and @samp{dim}.
> +@end table
> +
> +The style-able objects are:
> +@table @code
> +@kindex set style filename
> +@item filename
> +Control the styling of file names.
> +
> +@kindex set style function
> +@item function
> +Control the styling of function names. These are managed with the
> +@code{set style function} family of commands.
> +
> +@kindex set style variable
> +@item variable
> +Control the styling of variable names. These are managed with the
> +@code{set style variable} family of commands.
> +
> +@kindex set style address
> +@item address
> +Control the styling of addresses. These are managed with the
> +@code{set style address} family of commands.
> +@end table

The text is OK, but you have numerous "set style" index entries that
all start with the same substring and will point to almost the same
page. This is not useful, IME. I suggest just a single index entry
about "set style" instead.

Thanks.