On Mon, Apr 23, 2018 at 10:48:39AM +0200, Rafael J. Wysocki wrote: > On Saturday, April 14, 2018 6:10:55 AM CEST Yu Chen wrote: > > From: Chen Yu <[email protected]> > > > > There's a use case during test to only print specific round of iterations > > if --iterations is specified, for example, with this patch applied: > > > > turbostat -i 5 -r 4 > > will capture 4 samples with 5 seconds interval. > > > > Cc: Len Brown <[email protected]> > > Cc: Rafael J Wysocki <[email protected]> > > Cc: Artem Bityutskiy <[email protected]> > > Cc: Doug Smythies <[email protected]> > > Cc: [email protected] > > Cc: [email protected] > > Signed-off-by: Chen Yu <[email protected]> [cut]... > > + if (iterations && (++done_iters >= iterations)) > > The inner parens are not needed here (and similarly below). > > If you fix this little one, please feel free to add > > Reviewed-by: Rafael J. Wysocki <[email protected]> > > to the patch (FWIW). > OK, will do, thanks! Yu
