[Lldb-commits] [PATCH] D60022: [Process] Fix WriteMemory return value

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 29 20:57:16 PDT 2019


JDevlieghere added a comment.

Thanks Ismail, the patch LGTM, assuming I understand the code correctly.

FWIW I think this function is a good example of how early returns might prevent something like this from happening in the future. If you agree and have the bandwidth, feel free to refactor it, either as part of this patch or as a separate patch.


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

https://reviews.llvm.org/D60022





More information about the lldb-commits mailing list