<div dir="ltr">Seems reasonable.  lgtm</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 24, 2015 at 3:18 PM Jason Molenda <<a href="mailto:jmolenda@apple.com">jmolenda@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jasonmolenda created this revision.<br>
jasonmolenda added a reviewer: zturner.<br>
jasonmolenda added a subscriber: lldb-commits.<br>
jasonmolenda set the repository for this revision to rL LLVM.<br>
<br>
I'm adding some new host logging to help diagnose problems of finding executable files or dSYMs, Greg suggested we should add it to the Host log channel.  Currently Socket::Read and Socket::Write are logging to Host which is super verbose.  He suggests that we should switch these over the Communication log channel instead.  Some of Socket.cpp is logging to LOG_CONNECTION already.  Zachary, any opinion on this?  I think you touched this most recently.  Not sure if you're using this logging yourself.<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11497&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=JUZe8qFUDS9fDu-7XpAlMi8zXjD1FpgSpRSQzJYEQGM&s=Rlhz4cCIowxk1tt0vrNnaBnlMVeG_OEDjasM5oQv3b4&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11497</a><br>
<br>
Files:<br>
  source/Host/common/Socket.cpp<br>
<br>
Index: source/Host/common/Socket.cpp<br>
===================================================================<br>
--- source/Host/common/Socket.cpp<br>
+++ source/Host/common/Socket.cpp<br>
@@ -139,7 +139,7 @@<br>
     NativeSocket sock = kInvalidSocketValue;<br>
     Error error;<br>
<br>
-    Log *log(lldb_private::GetLogIfAnyCategoriesSet (LIBLLDB_LOG_HOST));<br>
+    Log *log(lldb_private::GetLogIfAnyCategoriesSet (LIBLLDB_LOG_COMMUNICATION));<br>
     if (log)<br>
         log->Printf ("Socket::TcpConnect (host/port = %s)", host_and_port.data());<br>
<br>
@@ -632,7 +632,7 @@<br>
     else<br>
         num_bytes = bytes_received;<br>
<br>
-    Log *log(lldb_private::GetLogIfAnyCategoriesSet (LIBLLDB_LOG_HOST | LIBLLDB_LOG_COMMUNICATION));<br>
+    Log *log(lldb_private::GetLogIfAnyCategoriesSet (LIBLLDB_LOG_COMMUNICATION));<br>
     if (log)<br>
     {<br>
         log->Printf ("%p Socket::Read() (socket = %" PRIu64 ", src = %p, src_len = %" PRIu64 ", flags = 0) => %" PRIi64 " (error = %s)",<br>
@@ -674,7 +674,7 @@<br>
     else<br>
         num_bytes = bytes_sent;<br>
<br>
-    Log *log(lldb_private::GetLogIfAnyCategoriesSet (LIBLLDB_LOG_HOST));<br>
+    Log *log(lldb_private::GetLogIfAnyCategoriesSet (LIBLLDB_LOG_COMMUNICATION));<br>
     if (log)<br>
     {<br>
         log->Printf ("%p Socket::Write() (socket = %" PRIu64 ", src = %p, src_len = %" PRIu64 ", flags = 0) => %" PRIi64 " (error = %s)",<br>
<br>
<br>
</blockquote></div>