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 01/13] constify cli-dump.c


Looks good.

Thanks,
Pedro Alves

On 07/22/2014 07:55 PM, Tom Tromey wrote:
> This does some minor constification in cli-dump.c.
> 
> 2014-07-22  Tom Tromey  <tromey@redhat.com>
> 
> 	* cli/cli-dump.c (scan_expression_with_cleanup): Return const.
> 	Make "cmd" const.
> 	(scan_filename_with_cleanup): Likewise.
> 	(dump_memory_to_file, dump_value_to_file, restore_binary_file):
> 	Make arguments const.
> 	(restore_command): Update.
> ---
>  gdb/ChangeLog      |  9 +++++++++
>  gdb/cli/cli-dump.c | 35 ++++++++++++++++++-----------------
>  2 files changed, 27 insertions(+), 17 deletions(-)
> 
> diff --git a/gdb/cli/cli-dump.c b/gdb/cli/cli-dump.c
> index aec732e..7929dc1 100644
> --- a/gdb/cli/cli-dump.c
> +++ b/gdb/cli/cli-dump.c
> @@ -35,8 +35,8 @@
>  #include "filestuff.h"
>  
>  
> -static char *
> -scan_expression_with_cleanup (char **cmd, const char *def)
> +static const char *
> +scan_expression_with_cleanup (const char **cmd, const char *def)
>  {
>    if ((*cmd) == NULL || (**cmd) == '\0')
>      {
> @@ -48,19 +48,19 @@ scan_expression_with_cleanup (char **cmd, const char *def)
>    else
>      {
>        char *exp;
> -      char *end;
> +      const char *end;
>  
>        end = (*cmd) + strcspn (*cmd, " \t");
>        exp = savestring ((*cmd), end - (*cmd));
>        make_cleanup (xfree, exp);
> -      (*cmd) = skip_spaces (end);
> +      (*cmd) = skip_spaces_const (end);
>        return exp;
>      }
>  }
>  
>  
>  static char *
> -scan_filename_with_cleanup (char **cmd, const char *defname)
> +scan_filename_with_cleanup (const char **cmd, const char *defname)
>  {
>    char *filename;
>    char *fullname;
> @@ -78,13 +78,13 @@ scan_filename_with_cleanup (char **cmd, const char *defname)
>    else
>      {
>        /* FIXME: should parse a possibly quoted string.  */
> -      char *end;
> +      const char *end;
>  
> -      (*cmd) = skip_spaces (*cmd);
> +      (*cmd) = skip_spaces_const (*cmd);
>        end = *cmd + strcspn (*cmd, " \t");
>        filename = savestring ((*cmd), end - (*cmd));
>        make_cleanup (xfree, filename);
> -      (*cmd) = skip_spaces (end);
> +      (*cmd) = skip_spaces_const (end);
>      }
>    gdb_assert (filename != NULL);
>  
> @@ -206,16 +206,16 @@ dump_bfd_file (const char *filename, const char *mode,
>  }
>  
>  static void
> -dump_memory_to_file (char *cmd, char *mode, char *file_format)
> +dump_memory_to_file (const char *cmd, const char *mode, const char *file_format)
>  {
>    struct cleanup *old_cleanups = make_cleanup (null_cleanup, NULL);
>    CORE_ADDR lo;
>    CORE_ADDR hi;
>    ULONGEST count;
> -  char *filename;
> +  const char *filename;
>    void *buf;
> -  char *lo_exp;
> -  char *hi_exp;
> +  const char *lo_exp;
> +  const char *hi_exp;
>  
>    /* Open the file.  */
>    filename = scan_filename_with_cleanup (&cmd, NULL);
> @@ -262,11 +262,11 @@ dump_memory_command (char *cmd, char *mode)
>  }
>  
>  static void
> -dump_value_to_file (char *cmd, char *mode, char *file_format)
> +dump_value_to_file (const char *cmd, const char *mode, const char *file_format)
>  {
>    struct cleanup *old_cleanups = make_cleanup (null_cleanup, NULL);
>    struct value *val;
> -  char *filename;
> +  const char *filename;
>  
>    /* Open the file.  */
>    filename = scan_filename_with_cleanup (&cmd, NULL);
> @@ -503,7 +503,7 @@ restore_section_callback (bfd *ibfd, asection *isec, void *args)
>  }
>  
>  static void
> -restore_binary_file (char *filename, struct callback_data *data)
> +restore_binary_file (const char *filename, struct callback_data *data)
>  {
>    struct cleanup *cleanup = make_cleanup (null_cleanup, NULL);
>    FILE *file = fopen_with_cleanup (filename, FOPEN_RB);
> @@ -555,12 +555,13 @@ restore_binary_file (char *filename, struct callback_data *data)
>  }
>  
>  static void
> -restore_command (char *args, int from_tty)
> +restore_command (char *args_in, int from_tty)
>  {
>    char *filename;
>    struct callback_data data;
>    bfd *ibfd;
>    int binary_flag = 0;
> +  const char *args = args_in;
>  
>    if (!target_has_execution)
>      noprocess ();
> @@ -580,7 +581,7 @@ restore_command (char *args, int from_tty)
>  	{
>  	  binary_flag = 1;
>  	  args += strlen (binary_string);
> -	  args = skip_spaces (args);
> +	  args = skip_spaces_const (args);
>  	}
>        /* Parse offset (optional).  */
>        if (args != NULL && *args != '\0')
> 


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