I can see both sides.  On the one hand, it is pretty unusual.  On the other hand, the change will probably have a large impact.  I guess if someone is willing to put in the work to fix the issue and all associated fallout then why not?  But there may also be external users of LLDB whom this will break.  <br><div><br></div><div>Also worth pointing out that we use connection strings in other places.  ConnectionFileDescriptor comes to mind.  Would need to audit those as well to make sure they are fixed.</div><div><br></div><div>If the change is going to be purely cosmetic, the risk / impact might not be worth the benefit, and it may just be better to add documentation about how the connection string works to the "lldb-platform --help" and "help platform connect".  </div><div><br></div><div>But I could at least conceive of ways which it could be useful.  For example, one potential benefit of Vince's approach is that you could set it up to listen for a connection from a set of IP addresses by specifying a list of remote hosts to --allow.</div><br><div class="gmail_quote">On Wed Jan 07 2015 at 9:46:12 AM Greg Clayton <<a href="mailto:gclayton@apple.com">gclayton@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I personally would rather not change this, everything else (lldb-gdbserver, debugserver, the GDB version of gdbserver) already accepts arguments or options as host:port.<br>
<br>
I would be interested in hearing everyone's opinion on the matter to make sure I am not alone.<br>
<br>
Greg<br>
<br>
<br>
> On Jan 6, 2015, at 5:43 PM, Vince Harron <<a href="mailto:vharron@google.com" target="_blank">vharron@google.com</a>> wrote:<br>
><br>
> IIUC, lldb-platform's --listen parameter works like this<br>
><br>
> lldb-platform --listen <remote-host>:<local-port><br>
><br>
> Usually, host:port notation indicates a host:port pair on the same interface.  Mixing local and remote like this is very strange to me.<br>
><br>
> I would prefer something more like<br>
><br>
> lldb-platform --listen <local-port> --allow <remote-host><br>
><br>
> if they want host/ip restrictions<br>
><br>
> I can make these changes if everyone agrees.<br>
><br>
> --<br>
><br>
> Vince Harron |         Technical Lead Manager |        <a href="mailto:vharron@google.com" target="_blank">vharron@google.com</a> |    858-442-0868<br>
><br>
> ______________________________<u></u>_________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-dev</a><br>
<br>
<br>
______________________________<u></u>_________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-dev</a><br>
</blockquote></div>