Hi Jiri, On Sun, Jan 3, 2021 at 7:05 AM Jiri Olsa <jo...@kernel.org> wrote: > +int cmd_daemon(int argc, const char **argv) > +{ > + struct option daemon_options[] = { > + OPT_INCR('v', "verbose", &verbose, "be more verbose"), > + OPT_STRING(0, "config", &__daemon.config, > + "config file", "config file path"), > + OPT_STRING_OPTARG('x', "field-separator", &__daemon.csv_sep, > + "field separator", "print counts with custom > separator", ":"),
Oh, I didn't expect it's optional and default to ":" instead of ",". Thanks, Namhyung > + OPT_END() > + };