curdeius accepted this revision.
curdeius added a comment.
This revision is now accepted and ready to land.

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).
Adding a new language would be to disruptive and difficult, as well as hard to 
use in some cases (headers in C and C++ often share the same extension).



================
Comment at: clang/unittests/Format/FormatTest.cpp:9926
+            format("void new (link p);\n"
+                   "void delete(link p);\n"));
 }
----------------
Why there's no space after `delete` in the input?


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

Reply via email to