>>> On 06.12.18 at 09:06, <[email protected]> wrote: > @@ -477,7 +478,8 @@ unsigned long long parse_size_and_unit(const char *s, > const char **ps) > s1++; > break; > default: > - ret <<= 10; /* default to kB */ > + if ( *s1 && *s1 != no_size ) > + ret <<= 10; /* default to kB */ > break;
So did you figure anything wrong with simply special casing '%' here? '%' is a form of "unit", after all. Perhaps demanding "ps" to be non-NULL (to have some form of indication the caller will check the suffix char) might be reasonable. Jan _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
