On 11/22/2012 08:09 PM, ithamar.adema@xxxxxxxxxxxxxxxx wrote:
b190a54: activitymonitor: remove B_MAX_CPU_COUNT reference It has no use, since we don't know its value and the list of colors might be longer [...]
That is certainly true, thanks for correcting. > (for example, for ARM currently B_MAX_CPU_COUNT is
only 1). The modula operator later on makes sure we keep within the bounds of the kColors array anyway.
However, why would you want to set it to 1 for ARM? Even if we currently (or in the foreseeable future) do not support SMP, there is no real reason to restrict that one, especially because changing it is a binary compatibility issue.
For future API rework, we should try to get rid of it, though, and use getter functions that get a CPU number or ID.
Bye, Axel.