Re: [PATCH] Remove "prefix" argument from cp_type_print_method_args

[Simon Marchi <simark@simark.ca>]

On 2026-04-17 22:10, Tom Tromey wrote:
> The "prefix" argument to cp_type_print_method_args is only ever the
> empty string, so it can be removed.
> ---
>  gdb/c-typeprint.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/gdb/c-typeprint.c b/gdb/c-typeprint.c
> index 821b78becf4..e418aca56a4 100644
> --- a/gdb/c-typeprint.c
> +++ b/gdb/c-typeprint.c
> @@ -244,7 +244,7 @@ cp_type_print_derivation_info (struct ui_file *stream,
>  /* Print the C++ method arguments ARGS to the file STREAM.  */
>  
>  static void
> -cp_type_print_method_args (struct type *mtype, const char *prefix,
> +cp_type_print_method_args (struct type *mtype,
>  			   const char *varstring, int staticp,
>  			   struct ui_file *stream,
>  			   enum language language,
> @@ -255,8 +255,6 @@ cp_type_print_method_args (struct type *mtype, const char *prefix,
>    int varargs = mtype->has_varargs ();
>    int i;
>  
> -  fprintf_symbol (stream, prefix,
> -		  language_cplus, DMGL_ANSI);
>    fprintf_symbol (stream, varstring,
>  		  language_cplus, DMGL_ANSI);
>    gdb_puts ("(", stream);
> @@ -1164,7 +1162,6 @@ c_type_print_base_struct_union (struct type *type, struct ui_file *stream,
>  		  struct type *mtype = TYPE_FN_FIELD_TYPE (f, j);
>  
>  		  cp_type_print_method_args (mtype,
> -					     "",
>  					     method_name,
>  					     staticp,
>  					     stream, language,

Seems obvious.

Approved-By: Simon Marchi <simon.marchi@efficios.com>

Simon