Properly strip end-of-option blanks
authorJens Axboe <jens.axboe@oracle.com>
Mon, 8 Dec 2008 13:06:42 +0000 (14:06 +0100)
committerJens Axboe <jens.axboe@oracle.com>
Mon, 8 Dec 2008 13:06:42 +0000 (14:06 +0100)
We don't want to fail with

option_foo = 0

when

option_foo=0

works.

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

diff --git a/parse.c b/parse.c
index 44c02f6..f838a1f 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)++;
+               strip_blank_end(ret);
                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) {
-               fprintf(stderr, "Bad option %s\n", opt);
+               fprintf(stderr, "Bad option <%s>\n", opt);
                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) {
-               fprintf(stderr, "Bad option %s\n", tmp);
+               fprintf(stderr, "Bad option <%s>\n", tmp);
                free(tmp);
                return 1;
        }