Fix for > 16 CPUs #1

Open
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

No description provided.

EDITED: the next commit fixes the missing %d, i did not see that :)

Oups, well spotted ... I wrote thoses plugins back in 2006 and you are the first to notice this bug, many thanks !!!

Owner

vincentg commented Jan 12, 2012

I will pull it as soon as possible and maybe just replace the modified printf by:

printf ("Cpu %d load=%d%% \n", count, load[count]);

To avoid any non ASCII char

Many thanks !

Vincent.

Yeah, I was really just trying to commit the one change of the Parenthesis,
but the Github edit form changed the non-ASCII character in the first
commit, so I fixed it up.

I've been using your plugin for a few years without any issues, but when I
tried on a system with 32 CPUs, it broke. :-)

Thanks for writing and distributing the plugins!

-Mark

On Thu, Jan 12, 2012 at 2:43 PM, Vincent Gerard <
reply@reply.github.com

wrote:

I will pull it as soon as possible and maybe just replace the modified
printf by:

printf ("Cpu %d load=%d%% \n", count, load[count]);

To avoid any non ASCII char

Many thanks !

Vincent.


Reply to this email directly or view it on GitHub:
#1 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment