intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-26 21:53:12 +08:00
Code Issues Packages Projects Releases Wiki Activity

[lldb] Fix -Wdangling-else warning in CommunicationTest

Browse Source 
Fixed warning: suggest explicit braces to avoid ambiguous ‘else’
[-Wdangling-else]
This commit is contained in:
Jonas Devlieghere
2023-03-08 22:21:00 -08:00
parent 73058e3301
commit f009241ae5
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
lldb/unittests/Core/CommunicationTest.cpp
View File

@@ -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);