[Lldb-commits] [lldb] r261356 - Fix Clang-tidy modernize-use-nullptr and modernize-use-default warnings in some files in source/Commands; other minor fixes.
Eugene Zelenko via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 19 13:03:28 PST 2016
On Fri, Feb 19, 2016 at 12:33 PM, Jim Ingham <jingham at apple.com> wrote:
> Please don't make this sort of change. With the code as originally written, I can set a breakpoint on the returns "true" or "false" case separately, and stop there if I am doing that experiment, whereas your change makes that impossible. Neither one is particularly harder to read than the other, but your version is significantly harder to debug.
>
> Jim
I this particular case it'w much easier to debug at point of usage.
Actually multiple comparisons could be replaced with strchr().
Eugene.
More information about the lldb-commits
mailing list