New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add readability-implicit-bool-conversion #12265
Add readability-implicit-bool-conversion #12265
Conversation
Unsurprisingly, this triggers hundreds of warnings :-) |
Yes, I rather wanted to quickly see. Where we are at. Obviously, also not relevant for the release. |
3366245
to
ca78945
Compare
I took another look at this. I would be in favor of merging the code changes for purposes of readability, and we can consider the changes to the clang-tidy flags separately. Want to drop the latter part and update the former and get this merged? |
Sure, I'll have a look. |
See #13150. |
ed62418
to
75eb125
Compare
75eb125
to
abecc20
Compare
@@ -2980,7 +2980,7 @@ inline FEEvaluationBase<dim, | |||
Assert(this->data == nullptr, ExcInternalError()); | |||
this->data = | |||
new internal::MatrixFreeFunctions::ShapeInfo<VectorizedArrayType>( | |||
Quadrature<dim - is_face>(quadrature), | |||
Quadrature<(is_face ? dim - 1 : dim)>(quadrature), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, this is so much more readable!
Similar to #12262 let's just make
clang-tidy
also check forint->bool
conversions.