Re: [Lldb-commits] [PATCH] D14937: swig_bot remote path preliminary implementation

2016-01-21 Thread Todd Fiala via lldb-commits
tfiala added a comment. Ready to close out? http://reviews.llvm.org/D14937 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Re: [Lldb-commits] [PATCH] D14937: swig_bot remote path preliminary implementation

2015-12-06 Thread Todd Fiala via lldb-commits
tfiala accepted this revision. tfiala added a comment. This revision is now accepted and ready to land. LGTM. http://reviews.llvm.org/D14937 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lld

Re: [Lldb-commits] [PATCH] D14937: swig_bot remote path preliminary implementation

2015-11-23 Thread Zachary Turner via lldb-commits
zturner updated this revision to Diff 40991. zturner added a comment. `socket.recv` doesn't guarantee a full read. The data can be truncated if it could not be read all at once. This patch fixes this by introducing a helper function in `lldbsuite.support` to read an exact number of bytes, retr