kadircet accepted this revision.
kadircet added a comment.
This revision is now accepted and ready to land.
thanks, lgtm!
================
Comment at: clang/lib/AST/StmtPrinter.cpp:2014
+ if (D->getInitStyle() == VarDecl::CallInit) {
+ if (!isa<ParenListExpr>(D->getInit())) {
+ Pre = "(";
----------------
ah, now i see the use for it, thanks for the testcase !
could you please `and` with the previous condition rather than extra nesting.
================
Comment at: clang/lib/AST/StmtPrinter.cpp:2018
+ }
+ } else if (D->getInitStyle() == VarDecl::CInit)
+ Pre = " = ";
----------------
nit: braces
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83855/new/
https://reviews.llvm.org/D83855
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
