EnterpriseDB/system_stats

Redundant logic.

Closed this issue · 2 comments

xianglin1998 commented

Is it normal to set the value of MaxClockSpeed to l2cache?
Although l2cache was set correctly afterwards.

system_stats/windows/cpu_info.c

Line 136 in 18033e7

hres = result->lpVtbl->Get(result, L"MaxClockSpeed", 0, &query_result, 0, 0);

neel5481 commented

yes, you are right, MaxClockSpeed is already getting read before L2CacheSize so removed the redundant code. Here is the PR ( #20 )

xianglin1998 commented

Perfect, thank you.