[ARI] Remove all editCase warnings

Daniel Jacobowitz dan@codesourcery.com
Wed May 5 18:34:00 GMT 2010


On Wed, May 05, 2010 at 09:55:29AM -0700, 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)
> 
> This applies to the rest of the patch, where we have the comment
> embedded right in the middle of parameter declaration... For instance:

Frankly, I'd rather we just dropped this ARI rule.

-- 
Daniel Jacobowitz
CodeSourcery



More information about the Gdb-patches mailing list