[PATCH] Add readability-simplify-boolean-expr check to clang-tidy
Richard
legalize at xmission.com
Sun Feb 15 18:04:08 PST 2015
Thinking about this a little more, I'm conflicted as to whether or not it should allow the simplification if the boolean constant came from a macro. If the macro is a feature detection macro, then I'm enabling/disabling chunks of code at runtime based on the resolution of the feature detection and I shouldn't be so quick to simplify that to always true or always false.
http://reviews.llvm.org/D7648
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list