Properly strip end-of-option blanks

We don't want to fail with

option_foo = 0

when

option_foo=0

works.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>

diff --git a/parse.c b/parse.c
index 44c02f60eb1488dba372ea34683bb4a3bba54e70..f838a1f625e111f6aa55a5bdf221a4f6352319a4 100644 (file)
--- a/parse.c
+++ b/parse.c
@@ -474,6 +474,7 @@ static struct fio_option *get_option(const char *opt,
                *ret = '\0';
                ret = (char *) opt;
                (*post)++;
                *ret = '\0';
                ret = (char *) opt;
                (*post)++;

+               strip_blank_end(ret);

                o = find_option(options, ret);
        } else {
                o = find_option(options, opt);
                o = find_option(options, ret);
        } else {
                o = find_option(options, opt);


@@ -520,7 +521,7 @@ int parse_cmd_option(const char *opt, const char *val,
 
        o = find_option(options, opt);
        if (!o) {
 
        o = find_option(options, opt);
        if (!o) {

-               fprintf(stderr, "Bad option %s\n", opt);
+               fprintf(stderr, "Bad option <%s>\n", opt);

                return 1;
        }
 
                return 1;
        }
 


@@ -589,7 +590,7 @@ int parse_option(const char *opt, struct fio_option *options, void *data)
 
        o = get_option(tmp, options, &post);
        if (!o) {
 
        o = get_option(tmp, options, &post);
        if (!o) {

-               fprintf(stderr, "Bad option %s\n", tmp);
+               fprintf(stderr, "Bad option <%s>\n", tmp);

                free(tmp);
                return 1;
        }
                free(tmp);
                return 1;
        }