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

LGTM modulo inline comments



================
Comment at: lldb/source/API/SBProcess.cpp:807-811
+  if (!dst) {
+    sb_error.SetErrorStringWithFormat(
+        "no buffer provided to read %zu bytes into", dst_len);
+    return 0;
+  }
----------------
Nit: this isn't using `bytes_read` so I'd move the new code above it.


================
Comment at: lldb/test/API/python_api/process/TestProcessAPI.py:70-72
+        if error.Success():
+            self.fail("SBProcessReadMemory claims to have "
+                      "successfully read 0xffffffffffffffe8 bytes")
----------------
`self.assertFalse(error.Success(), "SBProcessReadMemory claims to have 
successfully read 0xffffffffffffffe8 bytes")`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143012/new/

https://reviews.llvm.org/D143012

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to