[PATCH] D80079: [clang-format] [NFC] isCpp() is inconsistently used to mean both C++ and Objective C, add language specific isXXX() functions

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 21 09:41:47 PDT 2020


MyDeveloperDay added a comment.



> As @MyDeveloperDay said, I'd like it mean C++ only. I find it confusing that it means C++ or ObjC (even if the latter is a superset of the former). I'd rather see it spelt as `isCppOrObjC()` even if it's verbose but at least removes all confusion IMO.

I'm kind of with you on `isCppOrObjC()`


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

https://reviews.llvm.org/D80079





More information about the cfe-commits mailing list