[Lldb-commits] [lldb] [lldb][windows] bound ReadProcessMemory (PR #200230)

[via lldb-commits]

================
@@ -614,44 +614,82 @@ static std::optional<std::string> GetFileNameByLoadAddress(HANDLE hProcess,
   return dos_path;
 }
 
+// Determine how many bytes can be read at `addr` in `hProcess` before crossing
+// out of the committed memory region containing it. Returns 0 if the address is
+// not within a committed region.
+static SIZE_T BytesReadableAt(HANDLE hProcess, LPCVOID addr) {
----------------
Nerixyz wrote:

A separate refactor sounds good. It would be great if clang-tidy (and thus clangd) could warn on naming, but it's explicitly disabled in LLDB's `.clang-tidy` since it was first added.

https://github.com/llvm/llvm-project/pull/200230