penagos marked an inline comment as done. penagos added a comment. In D120374#3339554 <https://reviews.llvm.org/D120374#3339554>, @curdeius wrote:
> In D120374#3339337 <https://reviews.llvm.org/D120374#3339337>, @penagos wrote: > >> Though it's still unclear to me whether or not it'd make more sense to >> continue to pursue a change to introduce `C` as a language. Thoughts? > > I prefer your current approach (and there should be not that many cases that > we need a special treatment for C). Makes sense. I'll also need someone to commit on my behalf: - Luis Penagos - l...@penagos.co Thanks! ================ Comment at: clang/unittests/Format/FormatTest.cpp:9926 + format("void new (link p);\n" + "void delete(link p);\n")); } ---------------- curdeius wrote: > Why there's no space after `delete` in the input? Whoops, good catch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120374/new/ https://reviews.llvm.org/D120374 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
