Re: [ARI] Remove all editCase warnings

Joel Brobecker
2010-05-05T16:55:54Z

> 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:

> static void
> -replace_slotN_contents (char *bundle, long long instr, int slotnum)
> +replace_slotN_contents (char *bundle, /* ARI: editCase function */
> + long long instr, int slotnum)

What do others think?

--
Joel