[cfe-commits] [PATCH] Add support for Decl::print() with AccessSpecDecl instance

Will Wilson will at indefiant.com
Fri Aug 24 10:07:28 PDT 2012


Ah... I see support for testing DeclPrinter arrived a few days ago with
r162245. I'll attempt to update the patch accordingly!

- Will.

On 24 August 2012 18:58, Will Wilson <will at indefiant.com> wrote:

> Hi All,
>
> A very minor one here, but potentially useful for those that run into it.
> Calling print() on an AccessSpecDecl currently results in nothing being
> printed. The attached patch rectifies this omission in DeclPrinter.cpp.
>
> Note: AccessSpecDecl's printed as part of a CXXRecordDecl are printed as
> they're handled as a special case in DeclPrinter::VisitDeclContext() -
> probably due to the unusual indentation convention used by these
> declarations.
>
> No test cases I'm afraid as I couldn't find an obvious way of adding one -
> unless I've missed something. Anyhow, the change should be safe enough.
>
> - Will.
>



-- 
*Indefiant Ltd.*
*
*
Firsby Lodge, New Main Road, Scamblesby, Louth, Lincs LN11 9XH UK
*Tel: +44 20 8123 7663 England Registered No. 07936820 VAT No. **128556202*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120824/f3272547/attachment.html>


More information about the cfe-commits mailing list