MyDeveloperDay added a comment. In D80940#2071247 <https://reviews.llvm.org/D80940#2071247>, @curdeius wrote:
> LGTM. > > For a second, I thought that you could simplify the code by removing this > @try condition (and calling the function > `FormatTokenLexer::tryTransformTryUsageForC()` only if `isCppOnly`, but given > that Objective is a superset of C, it's probably safer to keep it the way > it's done now. this is the premise of D80079: [clang-format] [NFC] isCpp() is inconsistently used to mean both C++ and Objective C, add language specific isXXX() functions <https://reviews.llvm.org/D80079> but as we now are learning isCpp() is what is says on the tin! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80940/new/ https://reviews.llvm.org/D80940 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
