> -----Original Message-----
> From: Thomas Monjalon <tho...@monjalon.net>
> Sent: 16 March 2023 23:15
> To: Srikanth Yalavarthi <syalavar...@marvell.com>
> Cc: dev@dpdk.org; Shivah Shankar Shankar Narayan Rao
> <sshankarn...@marvell.com>; Jerin Jacob Kollanukkaran
> <jer...@marvell.com>; Anup Prabhu <apra...@marvell.com>; Prince Takkar
> <ptak...@marvell.com>; Parijat Shukla <pshu...@marvell.com>
> Subject: [EXT] Re: [PATCH v6 05/12] app/mldev: add ordered inference test
> case
>
> External Email
>
> ----------------------------------------------------------------------
> 11/03/2023 16:08, Srikanth Yalavarthi:
> > static struct option lgopts[] = {
> > - {ML_TEST, 1, 0, 0}, {ML_DEVICE_ID, 1, 0, 0}, {ML_SOCKET_ID, 1, 0, 0},
> {ML_MODELS, 1, 0, 0},
> > - {ML_DEBUG, 0, 0, 0}, {ML_HELP, 0, 0, 0}, {NULL, 0, 0, 0}};
> > + {ML_TEST, 1, 0, 0}, {ML_DEVICE_ID, 1, 0, 0}, {ML_SOCKET_ID, 1, 0,
> 0},
> > + {ML_MODELS, 1, 0, 0}, {ML_FILELIST, 1, 0, 0}, {ML_REPETITIONS, 1, 0,
> 0},
> > + {ML_DEBUG, 0, 0, 0}, {ML_HELP, 0, 0, 0}, {NULL, 0, 0, 0}};
>
> I don't understand the need to have multiple options per line.
> When doing a list, it is simpler to read on item per line.
This is a result of running checkformat / clang-format tool. The tool is
automatically arranging multiple options per line in the initial patches. The
final patch would have one option per line.
>
>
>
