> > -   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.

Reply via email to