Sounds good to me.  <br><br><div class="gmail_quote">On Tue Nov 11 2014 at 11:30:31 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On Nov 10, 2014, at 8:03 PM, Oleksiy Vyalov <<a href="mailto:ovyalov@google.com" target="_blank">ovyalov@google.com</a>> wrote:<br>
><br>
>>> ! In D6204#10, @zturner wrote:<br>
>> Can I also request that the variable *not* be called close_on_exec?.  It is<br>
>> possible to implement an equivalent on Windows, but calling it close on<br>
>> exec will be confusing to people on Windows.  Can we come up with a more<br>
>> generic name?  On Windows this would be called "inherit handles", which to<br>
>> me sounds like a better name than close on exec, but I may be biased.  Is<br>
>> "inherit handles" confusing to people on non-Windows?<br>
>><br>
>> In other words, I'm proposing<br>
>><br>
>> virtual lldb::ConnectionStatus ConnectionFileDescriptor::<u></u>Connect(const char<br>
>> *s, Error *error_ptr, bool close_on_exec = true);<br>
>><br>
>> would become<br>
>><br>
>> virtual lldb::ConnectionStatus ConnectionFileDescriptor::<u></u>Connect(const char<br>
>> *s, Error *error_ptr, bool inherit_handles = false);<br>
><br>
> It sounds good to me to call this variable inherit_handles in order to make it more platform-independent.<br>
<br>
Handles is a windows term and doesn't make sense for unix. How about "child_processes_inherit"?<br>
<br>
> As I can see ConnectionFileDescriptor::<u></u>Connect is overrides pure method from Connection::Connect (const char *url, Error *error_ptr)  and I'm a bit reluctant to make such variable visible for Communication layer  - so, I'm thinking about extending  ConnectionFileDescriptor constructor in a way like this:<br>
><br>
> ConnectionFileDescriptor(int fd, bool owns_fd, bool inherit_handles = false);<br>
><br>
> In this case methods within ConnectionFileDescriptor::<u></u>Connect (SocketListen, NamedSocketAccept, NamedSocketAccept, ConnectTCP, ConnectUDP) may use member field m_inherit_handles instead of taking additional argument.<br>
><br>
> <a href="http://reviews.llvm.org/D6204" target="_blank">http://reviews.llvm.org/D6204</a><br>
><br>
><br>
<br>
</blockquote></div>