xazax.hun accepted this revision.
xazax.hun added a comment.

Sorry for the delay, I think this is OK to commit.
As a possible improvement, I can imagine making it slightly stricter, e.g. you 
could only skip QualifiedNames for inline namespaces and the beginning. Maybe 
add support for staring with `""` or `::` to even disable skipping namespaces 
at the beginning?
But these are just nice to have features, I am perfectly ok with not having 
them or doing it in a followup patch.


https://reviews.llvm.org/D48027



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to