[clang-tools-extra] r245548 - [clang-tidy] Fold the meat of the UseNullPtrCheck into an anonymous namespace.

[Alexander Kornienko via cfe-commits]

Thanks, Benjamin!

On Thu, Aug 20, 2015 at 11:47 AM, Benjamin Kramer via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: d0k
> Date: Thu Aug 20 04:47:06 2015
> New Revision: 245548
>
> URL: http://llvm.org/viewvc/llvm-project?rev=245548&view=rev
> Log:
> [clang-tidy] Fold the meat of the UseNullPtrCheck into an anonymous
> namespace.
>
> While convenient, RecursiveASTVisitor generates a ridiculous amount of dead
> template code. Making it not visible from the outside lets the compiler
> eliminate some of it, shrinking clang-tidy by ~140k.
>
> Modified:
>     clang-tools-extra/trunk/clang-tidy/modernize/UseNullptrCheck.cpp
>
> Modified: clang-tools-extra/trunk/clang-tidy/modernize/UseNullptrCheck.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/modernize/UseNullptrCheck.cpp?rev=245548&r1=245547&r2=245548&view=diff
>
> ==============================================================================
> --- clang-tools-extra/trunk/clang-tidy/modernize/UseNullptrCheck.cpp
> (original)
> +++ clang-tools-extra/trunk/clang-tidy/modernize/UseNullptrCheck.cpp Thu
> Aug 20 04:47:06 2015
> @@ -20,6 +20,7 @@ using namespace llvm;
>  namespace clang {
>  namespace tidy {
>  namespace modernize {
> +namespace {
>
>  const char CastSequence[] = "sequence";
>
> @@ -438,6 +439,7 @@ private:
>    Expr *FirstSubExpr;
>    bool PruneSubtree;
>  };
> +} // namespace
>
>  UseNullptrCheck::UseNullptrCheck(StringRef Name, ClangTidyContext
> *Context)
>      : ClangTidyCheck(Name, Context),
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150820/3455a032/attachment.html>