tatyana-krasnukha added inline comments.
================ Comment at: source/Target/Process.cpp:2437 + for (auto &bp_site : enabled_bp_sites_in_range) + update_status(DisableSoftwareBreakpoint(bp_site.get())); ---------------- xiaobai wrote: > If disabling any breakpoint fails, the status's error string will be "failed > to re-enable one or more breakpoints". This is kind of confusing, imo. Didn't found anything better than use word "update" here. Introducing additional flag looks ugly... I hope, that this information is enough when user tries to write memory. More detailed description can be found in log. https://reviews.llvm.org/D39967 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
