JDevlieghere added inline comments.

================
Comment at: lldb/source/Commands/CommandObjectProcess.cpp:582
       }
-      
+
       Target *target = m_exe_ctx.GetTargetPtr();
----------------
wallace wrote:
> many formatting changes sneaked in. The actual changes are in the end of this 
> file
Please separate the formatting changes into a separate patch/commit. That way 
they're much easier to identify in the git blame output instead of getting 
attributed to an unrelated change. What I usually do in this scenario is format 
the file, commit than, and then rebase my change on top. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129239

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

Reply via email to