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

[Simon Marchi <simark@simark.ca>]

On 2025-06-20 14:12, Pedro Alves wrote:
> 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.

Thanks, done.

Simon