clayborg added a comment.

I like the new UI! I think you are right in that we don't need the address on 
the breakpoint location line if we can expand it and see it in the location 
children, so lets remove it from there. Other than that it looks good. The only 
other suggestion I would have is if we should display "1.1" and "1.2" for the 
breakpoint locations. This will match the output of "breakpoint list" a bit 
better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107386

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to