owenpan added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:2665
+    if (Style.isVerilog() && Precedence == prec::Comma &&
+        VerilogFirstOfType != nullptr) {
+      addFakeParenthesis(VerilogFirstOfType, prec::Comma);
----------------
sstwcw wrote:
> owenpan wrote:
> > And other places as well.
> @HazardyKnusperkeks Do you agree?  I remember you once said you preferred the 
> `nullptr` style.
See D144355. Not using `nullptr` in conditionals was one of the first LLVM 
styles I had to get used to when I started contributing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143825

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

Reply via email to