This revision was automatically updated to reflect the committed changes. Closed by commit rG9846a1f2d472: [lldb] Remove an outdated comment. NFC. (authored by mstorsjo).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128226/new/ https://reviews.llvm.org/D128226 Files: lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp Index: lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp =================================================================== --- lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp +++ lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp @@ -281,10 +281,6 @@ SIZE_T num_of_bytes_read = 0; if (!::ReadProcessMemory(process.GetNativeProcess().GetSystemHandle(), addr, buf, size, &num_of_bytes_read)) { - // Reading from the process can fail for a number of reasons - set the - // error code and make sure that the number of bytes read is set back to 0 - // because in some scenarios the value of bytes_read returned from the API - // is garbage. error.SetError(GetLastError(), eErrorTypeWin32); LLDB_LOG(log, "reading failed with error: {0}", error); } else {
Index: lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp =================================================================== --- lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp +++ lldb/source/Plugins/Process/Windows/Common/ProcessDebugger.cpp @@ -281,10 +281,6 @@ SIZE_T num_of_bytes_read = 0; if (!::ReadProcessMemory(process.GetNativeProcess().GetSystemHandle(), addr, buf, size, &num_of_bytes_read)) { - // Reading from the process can fail for a number of reasons - set the - // error code and make sure that the number of bytes read is set back to 0 - // because in some scenarios the value of bytes_read returned from the API - // is garbage. error.SetError(GetLastError(), eErrorTypeWin32); LLDB_LOG(log, "reading failed with error: {0}", error); } else {
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits