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

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Mon Nov 23 15:27:14 PST 2015


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, retrying as necessary until it's complete.  Then both the client and server are updated to use this function instead of `socket.recv`


http://reviews.llvm.org/D14937

Files:
  packages/Python/lldbsuite/support/sockutil.py
  scripts/swig_bot.py
  scripts/swig_bot_lib/client.py
  scripts/swig_bot_lib/local.py
  scripts/swig_bot_lib/server.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14937.40991.patch
Type: text/x-patch
Size: 14661 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151123/6a1859d7/attachment.bin>


More information about the lldb-commits mailing list