This is the mail archive of the gdb-patches@sources.redhat.com 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]

Re: [{AT


> Date: Thu, 3 Aug 2000 12:09:41 -0700
> From: Kevin Buettner <kevinb@cygnus.com>
> 
> Since indent seems to have difficulty with comments in the parameter
> list, I suspect that this isn't the only case that I'll have to
> override indent on.  (But if the result looks at all reasonable, I'll
> prefer indent output over my own hand formatting.)
[snip]
>  #if 0
>  static tree
> -parse_opt_name_string (allow_all)
> -     int allow_all;		/* 1 if ALL is allowed as a postfix */
> +parse_opt_name_string (int allow_all /* 1 if ALL is allowed as a postfix */ )

FWIW, I think such cases should be edited to move the comment before
the function definition.  For example:

  /* Parse the name of an option string.  If ALLOW_ALL is 1, ALL is
     allowed as a postfix */
  parse_opt_name_string (int allow_all)
  {

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