[Lldb-commits] [PATCH] D19533: Introduce Connection::ReadAll and fix AdbClient

[Pavel Labath via lldb-commits]

labath added inline comments.

================
Comment at: source/Plugins/Platform/Android/AdbClient.cpp:37
@@ -36,3 +36,3 @@
 
-const uint32_t kReadTimeout = 1000000; // 1 second
+const uint32_t kReadTimeout = 4000000; // 4 seconds
 const char * kOKAY = "OKAY";
----------------
ovyalov wrote:
> It might be useful to expose this timeout as a settings property to allow its customization for slow ADB connections.
We can do it later if a need appears, but I don't see how making that a setting --  the value needs to be big enough to work under all circumstances, and in case of normal operation, it will not matter, as the function will return as soon as the data is ready.


http://reviews.llvm.org/D19533