Re: [PATCH v2] Improve portability: Cast pid_t's to intmax_t

This seems rather pointless. Whatever Solaris thinks, I really doubt that process IDs will ever overflow an int, which is 32 bits on all machines that will ever support more than 32k processes.

You can be paranoid if you like and cast to long, but I don't think even a massive 64-bit cluster is likely to have more than 2G processes in the forseeable future.

I'd support a cast to int or unsigned, or a cast to long as a second choice.

Using uintmax_t is formally correct, but practically pointless clutter.