PiotrZSL added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/performance/FasterStringFindCheck.cpp:40 + // "'" is OK, but ''' is not, so add a backslash + if (ClosePos - OpenPos == 2 && Result[OpenPos + 1] == '\'') + Result.replace(OpenPos + 1, 1, "\\'"); ---------------- ================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:199-201 +- Fixed a bug in :doc:`performance-faster-string-find + <performance-faster-string-find>` which generated incorrect fixes by not + escaping single quotes properly. ---------------- And put it before performanc-noexcept-swap, other alternative would be to call it "Improve XYZ check to properly escape single quote characters. or something like that Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157436/new/ https://reviews.llvm.org/D157436 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits