JDevlieghere added a comment.

In D75810#1912248 <https://reviews.llvm.org/D75810#1912248>, @labath wrote:

> Yeah, naming conventions in lldb have always been a controversial topic. Most 
> of the code uses the lldb convention, but some of the new code (particularly 
> if it interfaces with llvm a lot) uses more llvm-like conventions. I think we 
> used to have a description of the naming convention somewhere on the website. 
> That is no longer there, but neither do we have a plan for migration.
>
> I don't really know what that means for this patch...


I would exclude `readability-identifier-naming` from the list of checks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75810



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

Reply via email to