> -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: 16 March 2023 23:32 > To: Srikanth Yalavarthi <[email protected]> > Cc: [email protected]; Shivah Shankar Shankar Narayan Rao > <[email protected]>; Jerin Jacob Kollanukkaran > <[email protected]>; Anup Prabhu <[email protected]>; Prince Takkar > <[email protected]>; Parijat Shukla <[email protected]>; Srikanth > Yalavarthi <[email protected]> > Subject: Re: [EXT] Re: [PATCH v6 05/12] app/mldev: add ordered inference > test case > > 16/03/2023 18:47, Srikanth Yalavarthi: > > From: Thomas Monjalon <[email protected]> > > > 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. > > Please do one option per line in each patch.
Done, changes are part of v7. >
