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