On Mon, 22 Feb 2016, Corinna Vinschen wrote:
One is, for completeness it would be nice if you could add a
description to the git comment along the lines of your original
comment so we have a description in the log.
Sorry, can't parse this; git newbie here. Did you mean the 'git commit'
I'm doing to my private repository and the message associated with the
commit? And by "original comment" do you mean what I called the change
log in the text of my v2 email we're discussing (i.e., not the patch
attachment but the email body)?
The other point is:
+ long divisor = 100000; // the power of 10 bigger than PID_MAX
I've seen 6 digit PIDs. In fact, we're not that tight on space here
so we should err on the side of caution and leave room for the entire
possible size of a Windows PID. That's a LONG, 32 bit, 10 decimal
digits.
Yikes. I'd seen large 5-digit pids but could not find a definitive symbol
defining Windows' maximum pid value. So I will change divisor's init
value to 1000*1000*1000 which will allow the conversion loop to support
10-digit pids.
Other than that, the patch looks good to me.
Great! I'll follow up with Jon separately (to the list) on his comments.
..mark