Re: [OBV][PATCH] gdb: tui_win_name: Make parameter and result const.

[Simon Marchi <simon.marchi@ericsson.com>]

On 15-07-06 11:15 AM, Andrew Burgess wrote:
> I have pushed the following as obvious.
> 
> Thanks,
> Andrew
> 
> ---
> 
> This commit makes the parameter and the result for 'tui_win_name'
> constant.  There's one place in the code that is then updated as a
> result of this change.
> 
> gdb/ChangeLog:
> 
> 	* tui/tui-data.c (tui_partial_win_by_name): Window name is const.
> 	(tui_win_name): Make parameter and result const.
> 	* tui/tui-data.h (tui_win_name): Make parameter and result const.
> ---
>  gdb/ChangeLog      | 6 ++++++
>  gdb/tui/tui-data.c | 7 ++++---
>  gdb/tui/tui-data.h | 2 +-
>  3 files changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/gdb/ChangeLog b/gdb/ChangeLog
> index 5131927..4636653 100644
> --- a/gdb/ChangeLog
> +++ b/gdb/ChangeLog
> @@ -1,3 +1,9 @@
> +2015-07-06  Andrew Burgess  <andrew.burgess@embecosm.com>
> +
> +	* tui/tui-data.c (tui_partial_win_by_name): Window name is const.
> +	(tui_win_name): Make parameter and result const.
> +	* tui/tui-data.h (tui_win_name): Make parameter and result const.
> +
>  2015-07-06  Patrick Palka  <patrick@parcs.ath.cx>
>  
>  	* i386-tdep.c (i386_mpx_info_bounds): Don't call error, instead
> diff --git a/gdb/tui/tui-data.c b/gdb/tui/tui-data.c
> index ed42c8d..0c350d8 100644
> --- a/gdb/tui/tui-data.c
> +++ b/gdb/tui/tui-data.c
> @@ -384,7 +384,8 @@ tui_partial_win_by_name (char *name)
>  	{
>            if (tui_win_list[i] != 0)
>              {
> -              char *cur_name = tui_win_name (&tui_win_list[i]->generic);
> +              const char *cur_name =
> +		tui_win_name (&tui_win_list[i]->generic);
>  
>                if (strlen (name) <= strlen (cur_name)
>  		  && startswith (cur_name, name))
> @@ -399,8 +400,8 @@ tui_partial_win_by_name (char *name)
>  
>  
>  /* Answer the name of the window.  */
> -char *
> -tui_win_name (struct tui_gen_win_info *win_info)
> +const char *
> +tui_win_name (const struct tui_gen_win_info *win_info)
>  {
>    char *name = (char *) NULL;

Hi Andrew,

I think you might as well make this variable const.

Simon