This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Correct comment in mi_cmd_data_write_memory_bytes


> This confused me for a moment, so I thought it would be worth
> correcting.

Absolutely, thank you!

> gdb/Changelog:
> 
> 2014-02-10  Simon Marchi  <simon.marchi@ericsson.com>
> 
> 	* mi/mi-main.c: Fix comment in mi_cmd_data_write_memory_bytes.

Small nit: The name of the function should be written between parentheses.
Like so:

 	* mi/mi-main.c (mi_cmd_data_write_memory_bytes): Fix comment.

I will commit this change under the obvious rule. But should you
think you'll be contributing more patches, you - or your employer
if you are making these changes as part of your work - will need
to assign your changes to the FSF. I can send you the paperwork
to get the process started, if you like. Once this is in place,
you also qualifier for Write After Approval privileges, meaning
that you can push your patches yourself once approved.

> ---
>  gdb/mi/mi-main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c
> index 93574ca..161307a 100644
> --- a/gdb/mi/mi-main.c
> +++ b/gdb/mi/mi-main.c
> @@ -1766,7 +1766,7 @@ mi_cmd_data_write_memory_bytes (char *command, char **argv, int argc)
>    else
>      {
>        /* Pattern is longer than or equal to count:
> -         just copy len bytes.  */
> +         just copy count bytes.  */
>        data = databuf;
>      }

-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]