mgartmann added a comment.
@aaron.ballman Thanks a lot for your valuable feedback! I incorporated it
accordingly.
Is there anything else that should be improved?
================
Comment at: clang-tools-extra/clang-tidy/google/ExplicitConstructorCheck.cpp:102
if (const auto *Conversion =
- Result.Nodes.getNodeAs<CXXConversionDecl>("conversion")) {
+ Result.Nodes.getNodeAs<CXXConversionDecl>("conversion")) {
if (Conversion->isOutOfLine())
----------------
aaron.ballman wrote:
> Unintended formatting change?
This formatting change was done by `clang-format` when `git clang-format
HEAD~1` was run. Therefore, I assumed that it is now correctly formatted.
Am I wrong?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102779/new/
https://reviews.llvm.org/D102779
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
