Re: [ARI] Remove all editCase warnings

Daniel Jacobowitz
2010-05-05T18:34:51Z

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