Messages in this thread | | | Subject | Re: [PATCH][v2] tools/power turbostat: if --max_loop, print for specific time of loops | From | Artem Bityutskiy <> | Date | Wed, 11 Apr 2018 12:29:04 +0300 |
| |
On Tue, 2018-04-10 at 22:51 +0800, Yu Chen wrote: > + case 't': > + { > + int loops = strtod(optarg, NULL); > + > + if (loops <= 0) { > + fprintf(outf, "loops %d should be positive number\n", > + iterations); > + exit(2); > + } > + iterations = loops; > + }
What is the point of the additional {} scope and the 'loops' variable in it? You could use the 'iterations' variable directly and simplify the code.
Artem.
| |