HazardyKnusperkeks added inline comments.
================ Comment at: clang/lib/Format/Format.cpp:1699 ChromiumStyle.DerivePointerAlignment = false; + ChromiumStyle.InsertBraces = true; if (Language == FormatStyle::LK_ObjC) ---------------- MyDeveloperDay wrote: > This is an code modifying feature, we agreed that all code modifying features > would be off by default, opt in only Now the question arises if //default// simply only applies to `LLVMStyle`, since that's the //default// when nothing is stated, or if other styles are free to enable such features in their style //by default//. I'd say if chromium wants to do that, they should be allowed to. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147969/new/ https://reviews.llvm.org/D147969 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits