[cfe-commits] [PATCH][Review Request] - handle non-copy constructor initializers

[Ted Kremenek]

Applied!

r114235

On Sep 17, 2010, at 5:46 AM, Jim Goodnow II wrote:

> Just a fixup to my previous patch that handles non-copy constructor initializers in DeclPrinter.
> 
> - jim
> 
> Index: lib/AST/DeclPrinter.cpp
> ===================================================================
> --- lib/AST/DeclPrinter.cpp	(revision 113834)
> +++ lib/AST/DeclPrinter.cpp	(working copy)
> @@ -521,8 +521,11 @@
>   if (Expr *Init = D->getInit()) {
>     if (D->hasCXXDirectInitializer())
>       Out << "(";
> -    else if (!dyn_cast<CXXConstructExpr>(Init))
> -      Out << " = ";
> +    else {
> +        CXXConstructExpr *CCE = dyn_cast<CXXConstructExpr>(Init);
> +        if (!CCE || CCE->getConstructor()->isCopyConstructor())
> +          Out << " = ";
> +    }
>     Init->printPretty(Out, Context, 0, Policy, Indentation);
>     if (D->hasCXXDirectInitializer())
>       Out << ")";<DeclPrinter.patch>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits