From f009241ae595ae6db5a859e498f72fd7e5dc6ce8 Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere Date: Wed, 8 Mar 2023 22:21:00 -0800 Subject: [PATCH] [lldb] Fix -Wdangling-else warning in CommunicationTest MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixed warning: suggest explicit braces to avoid ambiguous ‘else’ [-Wdangling-else] --- lldb/unittests/Core/CommunicationTest.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lldb/unittests/Core/CommunicationTest.cpp b/lldb/unittests/Core/CommunicationTest.cpp index 19ca4cb3d249..df9ff089a0d7 100644 --- a/lldb/unittests/Core/CommunicationTest.cpp +++ b/lldb/unittests/Core/CommunicationTest.cpp @@ -86,8 +86,9 @@ static void CommunicationReadTest(bool use_read_thread) { // Test using Communication that is disconnected. ASSERT_EQ(comm.Disconnect(), lldb::eConnectionStatusSuccess); - if (use_read_thread) + if (use_read_thread) { ASSERT_TRUE(comm.StartReadThread()); + } error.Clear(); EXPECT_EQ( comm.Read(buf, sizeof(buf), std::chrono::seconds(5), status, &error), 0U); @@ -97,8 +98,9 @@ static void CommunicationReadTest(bool use_read_thread) { // Test using Communication without a connection. comm.SetConnection(nullptr); - if (use_read_thread) + if (use_read_thread) { ASSERT_TRUE(comm.StartReadThread()); + } error.Clear(); EXPECT_EQ( comm.Read(buf, sizeof(buf), std::chrono::seconds(5), status, &error), 0U);