16 Jan
2015
16 Jan
'15
10:32 p.m.
Pete Bartlett
It is defined but unused if you're Unix but not Linux or Sun, so maybe Cygwin can give you this.
I'd rather not overcomplicate the code because of this.
I guess you're concerned about overcomplicating code just to suppress some warnings. What I was thinking of really though is that the process_info class is currently silently doing nothing (i.e. populating parent_pid, binary_name, binary_path with zero or blank) in the Cygwin case, but would probably work (I haven't tested) if what is currently the Linux-only code path was used for Cygwin. Sure the code still isn't getting any less complicated, but at least you'd be adding functionality! And good to know that clang isn't buggy here after all :-) Pete