[PATCH] D80800: Add an option to fully qualify classes and structs.

2020-06-01 Thread Jean-Baptiste Lespiau via Phabricator via cfe-commits
jblespiau marked an inline comment as done. jblespiau added inline comments. Comment at: clang/lib/AST/TypePrinter.cpp:326 + if (Policy.FullyQualifiedName && Policy.GlobalScopeQualifiedName && + T->isStructureOrClassType()) { +OS << "::"; jblespiau wrot

[PATCH] D80800: Add an option to fully qualify classes and structs.

2020-05-31 Thread Jean-Baptiste Lespiau via Phabricator via cfe-commits
jblespiau marked an inline comment as done. jblespiau added a comment. I did spend a few hours, just building and finding how to run tests :( I have a few additional questions, as I do not really understand what happen. In my initial idea, I wanted to modify the way QualType are printed, not rea

[PATCH] D80800: Add an option to fully qualify classes and structs.

2020-05-29 Thread Jean-Baptiste Lespiau via Phabricator via cfe-commits
jblespiau created this revision. jblespiau added a reviewer: sammccall. jblespiau edited the summary of this revision. sammccall added a comment. This should really have a test - NamedDeclPrinterTest.cpp seems like the right place. Comment at: clang/lib/AST/TypePrinter.cpp:326