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: [ARI] Remove all editCase warnings



> -----Message d'origine-----
> De?: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] De la part de Stan Shebs
> Envoyé?: Wednesday, May 05, 2010 7:16 PM
> À?: Joel Brobecker
> Cc?: Pierre Muller; 'Pedro Alves'; gdb-patches@sourceware.org
> Objet?: Re: [ARI] Remove all editCase warnings
> 
> Joel Brobecker wrote:
> >>  static void
> >> -ada_remove_Xbn_suffix (const char *encoded, int *len)
> >> +ada_remove_Xbn_suffix ( /* ARI: editCase function */
> >> +		       const char *encoded, int *len)
> >>
> >
> > I would rather have:
> >
> > ada_remove_Xbn_suffix /* ARI: editCase function */
> >   (const char *encoded, int *len)
> >
> 
> I frequently grep for "foo (", so this would cause the function to be
> overlooked.  But, fewer grep results means less work when changing the
> code... :-)
  That is also what is done inside gdb_ari.sh,
which means that it is not possible to fulfill
Joel's request.

  Eliminating the ' (' in the pattern would probably lead to lots of
false positive.

  I don't know what to think now...

  I fully agree that this is not one of the important 
rules, but I hoped that clearing up more fields would make
the ARI more efficient.
  Simply removing the rule is not better than 
leaving the source code unchanged in my opinion,
but I will follow the consensus if we reach one.

  
Pierre



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