Re: [PATCH 23/23] Multi-target: NEWS and user manual

Eli Zaretskii
2019-09-07T06:33:43Z

> From: Pedro Alves <palves@redhat.com>
> Date: Sat, 7 Sep 2019 00:28:07 +0100
>
> -
> -* Inferiors and Programs:: Debugging multiple inferiors and programs
> +* Inferiors Connections and Programs:: Debugging multiple inferiors connections and programs

The new line is too long, please break it in two.

> +@item
> +the target connection the inferior is bound to. This includes the
> +unique connection number assigned by @value{GDBN}, and the name of the
> +protocol used by the connection.

Having this @item start with a lowercase letter, and then having
another sentence in it looks awkward, English-wise. How about making
this a single sentence, as in "... is bound to, including the unique
connection number ..."?

> +@enumerate
> +@item
> +the connection number assigned by @value{GDBN}

This should end in a period, as the other items do.

> +@item
> +the connection name, derived from the protocol used by the connection.

"Name" sounds ... inaccurate, when it can be something like
"extended-remote host:10000", doesn't it? How about "Type"? Or maybe
call that "Description" and the last field "Details"?

> +There's no explicit way to switch focus between connections. Instead,
> +you switch focus between inferiors, which implicitly switches the
> +focus to the selected inferior's connection.
> +
> To switch focus between inferiors, use the @code{inferior} command:

What does "focus" mean in this context? Either we should explain that
here, or have a cross-reference where that is already explained.

> diff --git a/gdb/NEWS b/gdb/NEWS
> index f382e887c0..20aad73aa5 100644
> --- a/gdb/NEWS
> +++ b/gdb/NEWS

This part is OK.

Thanks.