Attention is currently required from: Lance Zhao, Nico Huber, Johnny Lin, Angel Pons, Patrick Rudolph.
9 comments:
Commit Message:
Patch Set #13, Line 11: intertool
typo: inte*l*tool
Done
Patchset:
Thanks for the review!
File util/inteltool/cpu.c:
Patch Set #13, Line 2300: range_end != (unsigned int)(-1)
uh... […]
Yes, it looks better, thanks!
File util/inteltool/inteltool.c:
Patch Set #13, Line 520: range
How about naming this option `cpu-range`? This should avoid name clashes if anyone wants to add anot […]
Ok, I agree, let's use --cpu-range.
Patch Set #13, Line 590: (unsigned int)(-1)
UINT_MAX
Done
Patch Set #13, Line 666: if (!dump_coremsrs) {
AFAIUI, this will bail out if --range is specified before -M. […]
It seems to me that this check is unnecessary and there is no point in it. It just won't work if the -M option isn't set. This code block was removed in the last patch.
errno = 0;
if (strlen(optarg) == 0 || errno) {
What do you use `errno` for here?
Sorry for the inattention. This is not necessary here.
Patch Set #13, Line 676: sscanf(optarg, "%u-%u", &cores_range_start, &cores_range_end);
> The sscanf function returns the value of the macro EOF if an input failure occurs before any conve […]
Ok, this way is much better, thanks!
Patch Set #13, Line 681: } else if (cores_range_end == (unsigned int)(-1)) {
Please use the return value of `sscanf()` to handle this instead of repeating a magic number.
Done
To view, visit change 35919. To unsubscribe, or for help writing mail filters, visit settings.