Re: [boost] [test] clang warning

Subject: Re: [boost] [test] clang warning
From: Pete Bartlett (pete_at_[hidden])
Date: 2015-01-16 17:32:21

Pete Bartlett <pete <at> pcbartlett.com> writes:

>> 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