On Thu, Jan 06, 2022 at 12:54:54PM -0800, Ira Weiny wrote:
> On Mon, Jan 03, 2022 at 12:16:17PM -0800, Schofield, Alison wrote:
> > From: Alison Schofield <alison.schofi...@intel.com>
> >
> > The OPT_U64 define failed in check_vtype() with unknown 'u64' type.
> > Replace with 'unsigned long long' to make the OPT_U64 define usable.
>
> I feel like this should be the first patch in the series.
I felt like it was a fixup, that should go right before I use it.
Now that the -size parameter is getting changed to a string,
(next patch feedback), this isn't needed.
I'll drop this patch from the set and save it for trivial cleanup
day.
more below...
>
> >
> > Signed-off-by: Alison Schofield <alison.schofi...@intel.com>
> > ---
> > util/parse-options.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/util/parse-options.h b/util/parse-options.h
> > index 9318fe7..91b7932 100644
> > --- a/util/parse-options.h
> > +++ b/util/parse-options.h
> > @@ -124,7 +124,7 @@ struct option {
> > #define OPT_INTEGER(s, l, v, h) { .type = OPTION_INTEGER, .short_name
> > = (s), .long_name = (l), .value = check_vtype(v, int *), .help = (h) }
> > #define OPT_UINTEGER(s, l, v, h) { .type = OPTION_UINTEGER, .short_name
> > = (s), .long_name = (l), .value = check_vtype(v, unsigned int *), .help =
> > (h) }
> > #define OPT_LONG(s, l, v, h) { .type = OPTION_LONG, .short_name =
> > (s), .long_name = (l), .value = check_vtype(v, long *), .help = (h) }
> > -#define OPT_U64(s, l, v, h) { .type = OPTION_U64, .short_name =
> > (s), .long_name = (l), .value = check_vtype(v, u64 *), .help = (h) }
> > +#define OPT_U64(s, l, v, h) { .type = OPTION_U64, .short_name =
> > (s), .long_name = (l), .value = check_vtype(v, unsigned long long *), .help
> > = (h) }
>
> Why can't this be uint64_t?
I don't know. ULL worked so I didn't look further.
Is uint64_t more suitable?
>
> Ira
>
> > #define OPT_STRING(s, l, v, a, h) { .type = OPTION_STRING, .short_name
> > = (s), .long_name = (l), .value = check_vtype(v, const char **), (a), .help
> > = (h) }
> > #define OPT_FILENAME(s, l, v, a, h) { .type = OPTION_FILENAME, .short_name
> > = (s), .long_name = (l), .value = check_vtype(v, const char **), (a), .help
> > = (h) }
> > #define OPT_DATE(s, l, v, h) \
> > --
> > 2.31.1
> >
