Improve load command's help text
Commit Message
This fairly obvious patch adds usage text to the load command's help text.
Originally it did not have usage and mentioned things like FILE and OFFSET
without explaining how those should be passed in the command.
gdb/ChangeLog:
2017-01-05 Luis Machado <lgustavo@codesourcery.com>
* symfile (_initialize_symfile): Add usage text to the load command's
help text.
---
gdb/symfile.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Comments
On 2017-01-06 11:41, Luis Machado wrote:
> diff --git a/gdb/symfile.c b/gdb/symfile.c
> index 61b9e5b..0af1900 100644
> --- a/gdb/symfile.c
> +++ b/gdb/symfile.c
> @@ -3952,7 +3952,8 @@ that lies within the boundaries of this symbol
> file in memory."),
> c = add_cmd ("load", class_files, load_command, _("\
> Dynamically load FILE into the running program, and record its
> symbols\n\
> for access from GDB.\n\
> -A load OFFSET may also be given."), &cmdlist);
> +A load offset may also be given.\n\
> +Usage: load [FILE] [offset expression]"), &cmdlist);
For consistency, I think this would look better:
Usage: load [FILE] [OFFSET]
The fact that offset can be an expression can be specified in the text
just before.
On 01/06/2017 12:29 PM, Simon Marchi wrote:
> On 2017-01-06 11:41, Luis Machado wrote:
>> diff --git a/gdb/symfile.c b/gdb/symfile.c
>> index 61b9e5b..0af1900 100644
>> --- a/gdb/symfile.c
>> +++ b/gdb/symfile.c
>> @@ -3952,7 +3952,8 @@ that lies within the boundaries of this symbol
>> file in memory."),
>> c = add_cmd ("load", class_files, load_command, _("\
>> Dynamically load FILE into the running program, and record its
>> symbols\n\
>> for access from GDB.\n\
>> -A load OFFSET may also be given."), &cmdlist);
>> +A load offset may also be given.\n\
>> +Usage: load [FILE] [offset expression]"), &cmdlist);
>
> For consistency, I think this would look better:
>
> Usage: load [FILE] [OFFSET]
>
> The fact that offset can be an expression can be specified in the text
> just before.
That sounds good to me.
Thanks,
Luis
On 17-01-06 10:41:28, Luis Machado wrote:
> This fairly obvious patch adds usage text to the load command's help text.
>
> Originally it did not have usage and mentioned things like FILE and OFFSET
> without explaining how those should be passed in the command.
>
> gdb/ChangeLog:
>
> 2017-01-05 Luis Machado <lgustavo@codesourcery.com>
>
> * symfile (_initialize_symfile): Add usage text to the load command's
> help text.
Patch is good to me, with Simon's suggestion.
@@ -3952,7 +3952,8 @@ that lies within the boundaries of this symbol file in memory."),
c = add_cmd ("load", class_files, load_command, _("\
Dynamically load FILE into the running program, and record its symbols\n\
for access from GDB.\n\
-A load OFFSET may also be given."), &cmdlist);
+A load offset may also be given.\n\
+Usage: load [FILE] [offset expression]"), &cmdlist);
set_cmd_completer (c, filename_completer);
add_prefix_cmd ("overlay", class_support, overlay_command,