labath added a comment.

I like it.



================
Comment at: 
lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp:4368-4371
+namespace lldb_private {
+namespace process_gdb_remote {
+
+struct RemoteRegisterInfo {
----------------
`struct process_gdb_remote::RemoteRegisterInfo {` should work too. Putting the 
declaration in the header would also be ok.


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

https://reviews.llvm.org/D110025

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

Reply via email to