[cfe-dev] clang-tidy google-build-using-namespace check and std::.*literals

Martin via cfe-dev cfe-dev at lists.llvm.org
Fri Feb 12 09:10:37 PST 2016


Hello,

(I am not subscribed to cfe-dev so please keep me in CC.)

I would like to get some feedback on
https://llvm.org/bugs/show_bug.cgi?id=25594 . Patch here:
https://llvm.org/bugs/attachment.cgi?id=15324 .

More specifically, why does my matcher modification not make it so
that the check skips std::.*literals namespaces? Have I misunderstood
what regular expression dialect is used in LLVM? Am I not using the
matcher correctly?

When writing the patch I tried to see how matchers are used in other
places and also tried to verify the regexp dialect. If I remember
correctly, I could not find any documentation for this though. Or, at
least any documentation or uses of matchers in other places of the
code that indicated to me that this should not work.

Also, is this patch something that is wanted? As far as I know there
is no way to use the standard defined literals without "using".

Thanks!

--
Martin



More information about the cfe-dev mailing list