[Lldb-commits] [PATCH] D19533: Introduce Connection::ReadAll and fix AdbClient
Pavel Labath via lldb-commits
lldb-commits at lists.llvm.org
Thu Apr 28 03:37:59 PDT 2016
labath updated this revision to Diff 55397.
labath added a comment.
Added proper handling of the timeout after retries, and moved the functionality into
Connection::Read. I have called the parameter read_full_buffer, as I think that better reflects
its intended use. I did not want to add a default value for it, since default values on virtual
functions can produce unexpected results. I've also had to increase the timeout in AdbClient,
since now it actually had a meaning (it seems that ReadAll was spinning in a loop also in case
the server was sending no data at all). The patch has grown bigger, but this is mostly due to
me refactoring unit tests to promote code reuse.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 24467 bytes
Desc: not available
More information about the lldb-commits