JonasToth added inline comments.
================ Comment at: clang-tidy/cppcoreguidelines/AvoidGotoCheck.cpp:43 + + // Backward jumps are diagnosed in all language modes. Forward jumps + // are sometimes required in C to free resources or do other clean-up ---------------- aaron.ballman wrote: > I think that this check should be C++ only. C makes far more use of `goto`, > and backwards jumps are not always bad there (they don't have to consider > things like destructors or RAII like you do in C++). > > Esp since this is a check for the C++ core guidelines and HICPP (both are C++ > standards). Ok. I merged the matchers too. ================ Comment at: docs/ReleaseNotes.rst:67-68 +- New `cppcoreguidelines-avoid-goto + <http://clang.llvm.org/extra/clang-tidy/checks/cppcoreguidelines-avoid-goto.html>`_ check + ---------------- aaron.ballman wrote: > I think you should also add the HICPP changes as well, given that this check > also covers that rule. I think `Only forward jumps in nested loops are accepted.` covers it, but i reformulated it. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D41815 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits