> > - cl = cmdline_stdin_new(main_ctx, "testpmd> "); > > - if (cl == NULL) { > > + testpmd_cl = cmdline_stdin_new(main_ctx, "testpmd> "); > > + if (testpmd_cl == NULL) { > > return; > > } > > Style nit: don't need {} around single statement. > > > +static void > > +sigint_handler(__rte_unused int signum) { > > + if (signum == SIGINT || signum == SIGTERM) { > > signmum is used, so don't want __rte_unused >
Thanks :) Will fix these in the next version.