search

cyring / CoreFreq

CoreFreq : CPU monitoring and tuning software designed for 64-bit processors.
https://www.cyring.fr
GNU General Public License v2.0
1.97k stars 126 forks source link

simple fix. #371

Closed ihsinme closed 1 year ago

ihsinme commented 1 year ago

this memset still did not work correctly https://coliru.stacked-crooked.com/a/4a5cc41c93fda992. but it is useless because you are filling the whole array below.

cyring commented 1 year ago

this memset still did not work correctly https://coliru.stacked-crooked.com/a/4a5cc41c93fda992. but it is useless because you are filling the whole array below.

Is the issue fixed by this change ?

memset(D, 0, 2 * OCCURRENCES * sizeof(unsigned long long));
ihsinme commented 1 year ago

yes, sure.

only, it's redundant. the loop below has filled the entire array so its zeroing can be removed

cyring commented 1 year ago

Fix should be part of next commit in branch develop

ihsinme commented 1 year ago

https://github.com/cyring/CoreFreq/pull/372