Hi Akim,

> Well, my reasoning was the opposite :)
> - I prefer to see the doc first, like documenting comments before
>   a function
> - I was more concerned with new "active" members than new documenting
>   members.
> 
> But I'll change it to match your way.

Thanks. Still a but surprising:

> +  /* All the features of an argmatch group.  */                         \
> +  typedef struct                                                        \
> +  {                                                                     \
> +    const argmatch_##Name##_doc* docs;                                  \
> +    const argmatch_##Name##_arg* args;                                  \

I would have expected this to be reordered like this:

     /* All the features of an argmatch group.  */                         \
     typedef struct                                                        \
     {                                                                     \
       const argmatch_##Name##_arg* args;                                  \
       const argmatch_##Name##_doc* docs;                                  \

Rationale:
* So that all doc strings are close together in the struct.
* You prefer to see the doc first, but in the .texi that you have written you
  let the programmer define first argmatch_backup_args and then - considering
  the "groups of synonyms" - the argmatch_backup_docs.

Bruno


Reply via email to