> Okay; +1 on `char short_name' instead of `char *short_names'.  As
> Greg points out, for the rare exceptions, one can specify two
> apr_option_t structs.

> But, in that case, we don't need a short_name equiv field at all --
> we can just use the val field (as the code currently does), right?

The current code uses the val field for exactly one purpose:
determining what value to yield when a long option is matched.

I don't have any particular objection to overloading val to also be a
short name of the option, when it is a valid character.  I just want
to be clear that that is a change.

With that change, we have (with yet more renaming):

typedef struct apr_option_t {
    /** long option name, or NULL if option has no long name */
    const char *name;
    /** option letter, or a value greater than 255 if option has no letter */
    int optch;
    /** nonzero if option takes an argument */
    int has_arg;
} apr_option_t;

APR_DECLARE(apr_status_t) apr_getopt_long(apr_getopt_t *os,
                                          const apr_option_t *opts,
                                          int *optch, const char **optarg);