curdeius accepted this revision.
curdeius added a comment.

In D106112#2885604 <https://reviews.llvm.org/D106112#2885604>, @owenpan wrote:

> I just reviewed the differences between K&R C (circa 1978) and ANSI/ISO C 
> again and didn't see anything else that would impact clang-format, so a new 
> Standard enum value for C78 is not needed. Nevertheless, we can add a boolean 
> option e.g. C78ParameterDecl in the future if this patch causes regressions 
> for some users. WDYT?

Sounds reasonable.
Any way, you can go forward and land this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106112

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

Reply via email to