jlebar marked 2 inline comments as done. jlebar added a comment. Thank you for the review, @alexfh. I will commit with these changes.
================ Comment at: clang-tools-extra/clang-tidy/performance/TypePromotionInMathFnCheck.cpp:198 + Result.Context->getSourceManager().getFileID(Call->getLocStart()), + "cmath", /* IsAngled = */ true)) + D << *IncludeFixit; ---------------- alexfh wrote: > Remove all spaces inside the argument comment, it's more common that way and > clang-format understand it better. Done, but FWIW it seems that clang-format will not insert a linebreak between the comment and the arg even with the spaces. It is much more common in llvm, though, as you say. https://reviews.llvm.org/D27748 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
