From ceb0cc54f9d647c2fed3ae878ba91b95f9740777 Mon Sep 17 00:00:00 2001 From: Haojian Wu Date: Mon, 3 Jun 2019 08:14:15 +0000 Subject: [clang-tidy] Fix make-unique check to work in C++17 mode. Summary: Previously, we intended to omit the check fix to the case when constructor has any braced-init-list argument. But the HasListInitializedArgument was not correct to handle all cases (Foo(Bar{1, 2}) will return false in C++14 mode). This patch fixes it, corrects the tests, and makes the check to run at C++17 mode. Reviewers: gribozavr Subscribers: xazax.hun, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D62736 llvm-svn: 362361 --- clang-tools-extra/test/clang-tidy/modernize-make-unique.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'clang-tools-extra/test/clang-tidy/modernize-make-unique.cpp') diff --git a/clang-tools-extra/test/clang-tidy/modernize-make-unique.cpp b/clang-tools-extra/test/clang-tidy/modernize-make-unique.cpp index 3b7a3de43ac..2920596e67c 100644 --- a/clang-tools-extra/test/clang-tidy/modernize-make-unique.cpp +++ b/clang-tools-extra/test/clang-tidy/modernize-make-unique.cpp @@ -1,5 +1,5 @@ -// RUN: %check_clang_tidy -std=c++14 %s modernize-make-unique %t -- -- -I %S/Inputs/modernize-smart-ptr -// FIXME: Fix the checker to work in C++17 mode. +// RUN: %check_clang_tidy -std=c++14,c++17 %s modernize-make-unique %t -- -- -I %S/Inputs/modernize-smart-ptr +// FIXME: Fix the test code in C++2a mode. #include "unique_ptr.h" #include "initializer_list.h" @@ -455,10 +455,10 @@ void initialization(int T, Base b) { std::unique_ptr PJ2 = std::unique_ptr(new J(E{1, 2}, 1)); // CHECK-MESSAGES: :[[@LINE-1]]:28: warning: use std::make_unique instead - // CHECK-FIXES: std::unique_ptr PJ2 = std::make_unique(E{1, 2}, 1); + // CHECK-FIXES: std::unique_ptr PJ2 = std::unique_ptr(new J(E{1, 2}, 1)); PJ2.reset(new J(E{1, 2}, 1)); // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: use std::make_unique instead - // CHECK-FIXES: PJ2 = std::make_unique(E{1, 2}, 1); + // CHECK-FIXES: PJ2.reset(new J(E{1, 2}, 1)); std::unique_ptr PJ3 = std::unique_ptr(new J{ {1, 2}, 1 }); // CHECK-MESSAGES: :[[@LINE-1]]:28: warning: use std::make_unique instead @@ -469,10 +469,10 @@ void initialization(int T, Base b) { std::unique_ptr PJ4 = std::unique_ptr(new J{E{1, 2}, 1}); // CHECK-MESSAGES: :[[@LINE-1]]:28: warning: use std::make_unique instead - // CHECK-FIXES: std::unique_ptr PJ4 = std::make_unique(E{1, 2}, 1); + // CHECK-FIXES: std::unique_ptr PJ4 = std::unique_ptr(new J{E{1, 2}, 1}); PJ4.reset(new J{E{1, 2}, 1}); // CHECK-MESSAGES: :[[@LINE-1]]:7: warning: use std::make_unique instead - // CHECK-FIXES: PJ4 = std::make_unique(E{1, 2}, 1); + // CHECK-FIXES: PJ4.reset(new J{E{1, 2}, 1}); std::unique_ptr FF = std::unique_ptr(new Foo()); // CHECK-MESSAGES: :[[@LINE-1]]:29: warning: -- cgit v1.2.3