arphaman added a comment.

If nothing uses `getCXXScopeSpecifier` right now we can't really test it with a 
clang or c-index-test regression test. A completion unit test could work here. 
I don't think we actually have existing completion unit tests though, so you 
would have to create one from scratch. But if `getCXXScopeSpecifier` will be 
used in a follow up patch maybe it will be easier to commit this without a test 
together with the followup patch?


https://reviews.llvm.org/D40563



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

Reply via email to