From a040597dafec356698e87871de38ec6dc26f4515 Mon Sep 17 00:00:00 2001 From: midipix Date: Sun, 25 Dec 2016 12:22:05 -0500 Subject: argv.h: use an option vector rather than a direct pointer to the option table. --- src/driver/sfrt_driver_ctx.c | 19 +++++---- src/internal/argv/argv.h | 83 ++++++++++++++++++++++++--------------- src/internal/sofort_driver_impl.h | 2 + 3 files changed, 65 insertions(+), 39 deletions(-) diff --git a/src/driver/sfrt_driver_ctx.c b/src/driver/sfrt_driver_ctx.c index d065474..58cccd7 100644 --- a/src/driver/sfrt_driver_ctx.c +++ b/src/driver/sfrt_driver_ctx.c @@ -43,7 +43,7 @@ static uint32_t sfrt_argv_flags(uint32_t flags) static int sfrt_driver_usage( const char * program, const char * arg, - const struct argv_option * options, + const struct argv_option ** optv, struct argv_meta * meta) { char header[512]; @@ -52,7 +52,7 @@ static int sfrt_driver_usage( "Usage: %s [options] ...\n" "Options:\n", program); - argv_usage(stdout,header,options,arg); + argv_usage(stdout,header,optv,arg); argv_free(meta); return SFRT_USAGE; @@ -107,7 +107,7 @@ int sfrt_get_driver_ctx( { struct sfrt_driver_ctx_impl * ctx; struct sfrt_common_ctx cctx; - const struct argv_option * options; + const struct argv_option * optv[SFRT_OPTV_ELEMENTS]; struct argv_meta * meta; struct argv_entry * entry; size_t nunits; @@ -115,9 +115,9 @@ int sfrt_get_driver_ctx( (void)envp; - options = sfrt_default_options; + argv_optv_init(sfrt_default_options,optv); - if (!(meta = argv_get(argv,options,sfrt_argv_flags(flags)))) + if (!(meta = argv_get(argv,optv,sfrt_argv_flags(flags)))) return -1; nunits = 0; @@ -126,7 +126,7 @@ int sfrt_get_driver_ctx( cctx.drvflags = flags; if (!argv[1] && (flags & SFRT_DRIVER_VERBOSITY_USAGE)) - return sfrt_driver_usage(program,0,options,meta); + return sfrt_driver_usage(program,0,optv,meta); /* get options, count units */ for (entry=meta->entries; entry->fopt || entry->arg; entry++) { @@ -134,7 +134,7 @@ int sfrt_get_driver_ctx( switch (entry->tag) { case TAG_HELP: if (flags & SFRT_DRIVER_VERBOSITY_USAGE) - return sfrt_driver_usage(program,entry->arg,options,meta); + return sfrt_driver_usage(program,entry->arg,optv,meta); case TAG_VERSION: cctx.drvflags |= SFRT_DRIVER_VERSION; @@ -169,11 +169,14 @@ int sfrt_create_driver_ctx( const struct sfrt_common_ctx * cctx, struct sfrt_driver_ctx ** pctx) { + const struct argv_option * optv[SFRT_OPTV_ELEMENTS]; struct argv_meta * meta; struct sfrt_driver_ctx_impl * ctx; char * argv[] = {"sofort_driver",0}; - if (!(meta = argv_get(argv,sfrt_default_options,0))) + argv_optv_init(sfrt_default_options,optv); + + if (!(meta = argv_get(argv,optv,0))) return -1; if (!(ctx = sfrt_driver_ctx_alloc(meta,cctx,0))) diff --git a/src/internal/argv/argv.h b/src/internal/argv/argv.h index 848e0af..f9ff983 100644 --- a/src/internal/argv/argv.h +++ b/src/internal/argv/argv.h @@ -139,17 +139,21 @@ struct argv_ctx { #ifdef ARGV_DRIVER +static int argv_optv_init( + const struct argv_option[], + const struct argv_option **); + static const char * argv_program_name(const char *); static void argv_usage( FILE *, const char * header, - const struct argv_option[], + const struct argv_option **, const char * mode); static struct argv_meta * argv_get( char **, - const struct argv_option[], + const struct argv_option **, int flags); static void argv_free(struct argv_meta *); @@ -161,14 +165,30 @@ static void argv_free(struct argv_meta *); /* implementation of static functions */ /*------------------------------------*/ +static int argv_optv_init( + const struct argv_option options[], + const struct argv_option ** optv) +{ + const struct argv_option * option; + int i; + + for (option=options,i=0; option->long_name || option->short_name; option++) + optv[i++] = option; + + optv[i] = 0; + return i; +} + static const struct argv_option * argv_short_option( const char * ch, - const struct argv_option options[], + const struct argv_option ** optv, struct argv_entry * entry) { const struct argv_option * option; - for (option=options; option->long_name || option->short_name; option++) { + for (; *optv; optv++) { + option = *optv; + if (option->short_name == *ch) { entry->tag = option->tag; entry->fopt = true; @@ -181,15 +201,16 @@ static const struct argv_option * argv_short_option( static const struct argv_option * argv_long_option( const char * ch, - const struct argv_option options[], + const struct argv_option ** optv, struct argv_entry * entry) { const struct argv_option * option; const char * arg; size_t len; - for (option=options; option->long_name || option->short_name; option++) { - len = option->long_name ? strlen(option->long_name) : 0; + for (; *optv; optv++) { + option = *optv; + len = option->long_name ? strlen(option->long_name) : 0; if (len && !(strncmp(option->long_name,ch,len))) { arg = ch + len; @@ -226,7 +247,7 @@ static inline bool is_last_option(const char * arg) static inline bool is_hybrid_option( const char * arg, - const struct argv_option options[]) + const struct argv_option ** optv) { const struct argv_option * option; struct argv_entry entry; @@ -234,11 +255,11 @@ static inline bool is_hybrid_option( if (!is_short_option(arg)) return false; - if (!(option = argv_long_option(++arg,options,&entry))) + if (!(option = argv_long_option(++arg,optv,&entry))) return false; if (!(option->flags & ARGV_OPTION_HYBRID_SWITCH)) - if (argv_short_option(arg,options,&entry)) + if (argv_short_option(arg,optv,&entry)) return false; return true; @@ -265,20 +286,18 @@ static inline bool is_arg_in_paradigm(const char * arg, const char * paradigm) } static inline const struct argv_option * option_from_tag( - const struct argv_option options[], + const struct argv_option ** optv, int tag) { - const struct argv_option * option; - - for (option=options; option->short_name || option->long_name; option++) - if (option->tag == tag) - return option; + for (; *optv; optv++) + if (optv[0]->tag == tag) + return optv[0]; return 0; } static void argv_scan( char ** argv, - const struct argv_option options[], + const struct argv_option ** optv, struct argv_ctx * ctx, struct argv_meta * meta) { @@ -316,14 +335,14 @@ static void argv_scan( else if (is_last_option(ch)) fnoscan = true; - else if (!fshort && is_hybrid_option(ch,options)) + else if (!fshort && is_hybrid_option(ch,optv)) fhybrid = true; if (!fnoscan && !fhybrid && (fshort || is_short_option(ch))) { if (!fshort) ch++; - if ((option = argv_short_option(ch,options,&entry))) { + if ((option = argv_short_option(ch,optv,&entry))) { if (ch[1]) { ch++; fnext = false; @@ -370,7 +389,7 @@ static void argv_scan( } else if (!fnoscan && (fhybrid || is_long_option(ch))) { ch += (fhybrid ? 1 : 2); - if ((option = argv_long_option(ch,options,&entry))) { + if ((option = argv_long_option(ch,optv,&entry))) { val = ch + strlen(option->long_name); /* val[0] is either '=' (or ',') or '\0' */ @@ -584,7 +603,7 @@ static void argv_show_error(struct argv_ctx * ctx) } static void argv_show_status( - const struct argv_option options[], + const struct argv_option ** optv, struct argv_ctx * ctx, struct argv_meta * meta) { @@ -610,7 +629,7 @@ static void argv_show_status( fputs("\n\nparsed entries:\n",stderr); for (entry=meta->entries; entry->arg || entry->fopt; entry++) if (entry->fopt) { - option = option_from_tag(options,entry->tag); + option = option_from_tag(optv,entry->tag); short_name[0] = option->short_name; if (entry->fval) @@ -683,13 +702,13 @@ static struct argv_meta * argv_alloc(char ** argv, struct argv_ctx * ctx) static struct argv_meta * argv_get( char * argv[], - const struct argv_option options[], + const struct argv_option ** optv, int flags) { struct argv_meta * meta; struct argv_ctx ctx = {flags,ARGV_MODE_SCAN,0,0,0,0,0,0,0}; - argv_scan(argv,options,&ctx,0); + argv_scan(argv,optv,&ctx,0); if (ctx.errcode != ARGV_ERROR_OK) { ctx.program = argv_program_name(argv[0]); @@ -704,7 +723,7 @@ static struct argv_meta * argv_get( return 0; ctx.mode = ARGV_MODE_COPY; - argv_scan(meta->argv,options,&ctx,meta); + argv_scan(meta->argv,optv,&ctx,meta); if (ctx.errcode != ARGV_ERROR_OK) { ctx.program = argv[0]; @@ -716,7 +735,7 @@ static struct argv_meta * argv_get( } if (ctx.flags & ARGV_VERBOSITY_STATUS) - argv_show_status(options,&ctx,meta); + argv_show_status(optv,&ctx,meta); return meta; } @@ -736,9 +755,10 @@ static void argv_free(struct argv_meta * xmeta) static void argv_usage( FILE * file, const char * header, - const struct argv_option options[], + const struct argv_option ** options, const char * mode) { + const struct argv_option ** optv; const struct argv_option * option; bool fshort,flong,fboth; size_t len,optlen,desclen; @@ -771,10 +791,9 @@ static void argv_usage( if (header) fprintf(file,"%s",header); - option = options; - optlen = 0; + for (optlen=0,optv=options; *optv; optv++) { + option = *optv; - for (; option->short_name || option->long_name; option++) { /* indent + comma */ len = fboth ? sizeof(indent) + 1 : sizeof(indent); @@ -805,7 +824,9 @@ static void argv_usage( optlen &= (~(ARGV_TAB_WIDTH-1)); desclen = (optlen < width / 2) ? width - optlen : optlen; - for (option=options; option->short_name || option->long_name; option++) { + for (optv=options; *optv; optv++) { + option = *optv; + /* color */ if (fcolor) { color = (color == ccyan) ? cblue : ccyan; diff --git a/src/internal/sofort_driver_impl.h b/src/internal/sofort_driver_impl.h index d6d2e44..a9788f4 100644 --- a/src/internal/sofort_driver_impl.h +++ b/src/internal/sofort_driver_impl.h @@ -8,6 +8,8 @@ #include #include "argv/argv.h" +#define SFRT_OPTV_ELEMENTS 64 + extern const struct argv_option sfrt_default_options[]; enum app_tags { -- cgit v1.2.3