mikecrowe wrote: @5chmidti wrote: > It'll be 1 week+ before I am back at my main machine to take a better look at > the macro question. Maybe this can just be landed, without that distinction.
Thanks for letting me know and thanks for the review. I've pushed a new version to this PR with the typo you spotted fixed and removing the commit containing the debug output for exploring the macro detection (the version with the extra debug is at https://github.com/mikecrowe/clang-tidy-fmt/commits/mac/fix-use-std-print-macro-expansion-investigation/ if you still need it). I'll let you and @PiotrZSL decide whether you want to land this PR as it is since I don't have permission to do so myself. https://github.com/llvm/llvm-project/pull/97911 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
