Re: [PATCH] D12162: Remove reference.

2015-08-19 Thread Alexander Kornienko via cfe-commits
alexfh closed this revision. alexfh added a comment. Committed revision 245471. http://reviews.llvm.org/D12162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D12162: Remove reference.

2015-08-19 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. Thanks! LG http://reviews.llvm.org/D12162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/

Re: [PATCH] D12162: Remove reference.

2015-08-19 Thread Angel Garcia via cfe-commits
angelgarcia changed the visibility of this Differential Revision from "All Users" to "Public (No Login Required)". angelgarcia updated this revision to Diff 32576. angelgarcia added a comment. Use ArrayRef. http://reviews.llvm.org/D12162 Files: clang-tidy/modernize/UseNullptrCheck.cpp Index

Re: [PATCH] D12162: Remove reference.

2015-08-19 Thread Alexander Kornienko via cfe-commits
alexfh added inline comments. Comment at: clang-tidy/modernize/UseNullptrCheck.cpp:178 @@ -177,3 +177,3 @@ CastSequenceVisitor(ASTContext &Context, SmallVector UserNullMacros, ClangTidyCheck &check) Please use Array

[PATCH] D12162: Remove reference.

2015-08-19 Thread Angel Garcia via cfe-commits
angelgarcia created this revision. angelgarcia added a reviewer: alexfh. angelgarcia added subscribers: cfe-commits, klimek. angelgarcia changed the visibility of this Differential Revision from "Public (No Login Required)" to "All Users". Remove a reference where a value was intended (its being