Re: [PATCH v3 2/6] gdb/solib: fix formatting of "info linker-namespaces" error message

[Pedro Alves <pedro@palves.net>]

On 2025-06-16 20:33, Simon Marchi wrote:
> This patch is new in v3.
> 
> Add spaces after the first period and add a period at the end, resulting
> in:
> 
>     (gdb) info linker-namespaces
>     ❌️ Current inferior does not support linker namespaces.  Use "info sharedlibrary" instead.
> 
> Change-Id: Ib3f1647cedcdb68852a3c63df26ea3e6f791b1b1
> ---
>  gdb/solib.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/gdb/solib.c b/gdb/solib.c
> index 7721fd673ae3..90e7b906222c 100644

> --- a/gdb/solib.c
> +++ b/gdb/solib.c
> @@ -1163,8 +1163,8 @@ info_linker_namespace_command (const char *pattern, int from_tty)
>    /* This command only really makes sense for inferiors that support
>       linker namespaces, so we can leave early.  */
>    if (ops->num_active_namespaces == nullptr)
> -    error (_("Current inferior does not support linker namespaces." \
> -	     "Use \"info sharedlibrary\" instead"));
> +    error (_("Current inferior does not support linker namespaces.  " \
> +	     "Use \"info sharedlibrary\" instead."));
>  

You could remove the useless continuation backslash too, while at it.

Approved-By: Pedro Alves <pedro@palves.net>