aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRosen Penev <rosenp@gmail.com>2020-06-02 18:07:40 -0700
committerRosen Penev <rosenp@gmail.com>2020-11-07 12:27:26 -0800
commit9fa3491284af9a595cd46c6b230bcffb0ab8aec9 (patch)
treee938030fcc49419fca00d145f65ff587f9a68b01
parent17bcd4c8d69d3e5a0d11092938d30ac29aeb65e1 (diff)
downloadninja-9fa3491284af9a595cd46c6b230bcffb0ab8aec9.tar.gz
[clang-tidy] remove pointless string init
Found with readability-redundant-string-init Signed-off-by: Rosen Penev <rosenp@gmail.com>
-rw-r--r--.clang-tidy2
-rw-r--r--src/string_piece_util_test.cc4
2 files changed, 4 insertions, 2 deletions
diff --git a/.clang-tidy b/.clang-tidy
index 8968011..df4c1ed 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -3,9 +3,11 @@ Checks: '
,readability-avoid-const-params-in-decls,
,readability-non-const-parameter,
,readability-redundant-string-cstr,
+ ,readability-redundant-string-init,
'
WarningsAsErrors: '
,readability-avoid-const-params-in-decls,
,readability-non-const-parameter,
,readability-redundant-string-cstr,
+ ,readability-redundant-string-init,
'
diff --git a/src/string_piece_util_test.cc b/src/string_piece_util_test.cc
index b77ae84..61586dd 100644
--- a/src/string_piece_util_test.cc
+++ b/src/string_piece_util_test.cc
@@ -31,7 +31,7 @@ TEST(StringPieceUtilTest, SplitStringPiece) {
}
{
- string empty("");
+ string empty;
vector<StringPiece> list = SplitStringPiece(empty, ':');
EXPECT_EQ(list.size(), 1);
@@ -82,7 +82,7 @@ TEST(StringPieceUtilTest, JoinStringPiece) {
}
{
- string empty("");
+ string empty;
vector<StringPiece> list = SplitStringPiece(empty, ':');
EXPECT_EQ("", JoinStringPiece(list, ':'));