[Lldb-commits] [PATCH] D11150: [NativeProcessLinux] Integrate MainLoop

Tamas Berghammer tberghammer at google.com
Mon Jul 13 10:35:51 PDT 2015


tberghammer accepted this revision.
tberghammer added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: source/Plugins/Process/Linux/NativeProcessLinux.cpp:2729
@@ -3154,3 +2728,3 @@
 
-    return DoOperation([&] { return DoReadMemory(GetID(), addr, buf, size, bytes_read); });
+    return DoReadMemory(GetID(), addr, buf, size, bytes_read);
 }
----------------
Please inline this function call

================
Comment at: source/Plugins/Process/Linux/NativeProcessLinux.cpp:2743
@@ -3168,3 +2742,3 @@
 {
-    return DoOperation([&] { return DoWriteMemory(GetID(), addr, buf, size, bytes_written); });
+    return DoWriteMemory(GetID(), addr, buf, size, bytes_written);
 }
----------------
Please inline this function call


http://reviews.llvm.org/D11150







More information about the lldb-commits mailing list