LegalizeAdulthood added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-macro-to-enum.rst:21-22 + `,`, `-`, `+`, `*`, `/`, `%`, `&`, `|`, `^`, `<`, `>`, `<=`, `>=`, + `==`, `!=`, `||`, `&&`, `<<`, `>>` or `<=>`, and the ternary operator + `?:`. Parenthesized expressions are also recognized. This + recognizes the most valid expressions. In particular, expressions ---------------- aaron.ballman wrote: > Maybe we should also list the binary `?:` GNU extension and comma expressions? Yes, I need to update the docs on that and also call out the potential false positives explicitly. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124500/new/ https://reviews.llvm.org/D124500 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits