krasimir accepted this revision. krasimir added a comment. This revision is now accepted and ready to land.
This is very nice! Brings the C# handling of the colon closer to other languages. ================ Comment at: clang/lib/Format/TokenAnnotator.cpp:3499 const FormatToken &Left = *Right.Previous; - + if (Style.isCSharp()) { + if (Left.is(TT_CSharpNamedArgumentColon) || ---------------- nit: please put this as an else if into the "// Language-specific stuff" section below. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75244/new/ https://reviews.llvm.org/D75244 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits