commit 8a1a47dff92618878bff404b775a920bb32fbc82
author Steven Moreland <smoreland@google.com> Tue Sep 14 10:54:04 2021 -0700
committer Steven Moreland <smoreland@google.com> Tue Sep 14 11:22:36 2021 -0700
tree ea17b266c145730e613b85ad6f64aa80a9762bc3
parent a54c8618434a6ca4cf81c11054b507bf4d073680

libbinder: RPC tests - avoid goto

devinmoore@ kindly pointed out it wasn't necessary, and it simplifies
the code.

Bug: N/A
Test: binderRpcTest, binderRpcBenchark, binder_bpBinderFuzz
Change-Id: I6c35a256f8790d71df3848193688085126c60da7

libs/binder/tests/binderRpcTest.cpp

diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp
index a4e37ad..a9273d6 100644
--- a/libs/binder/tests/binderRpcTest.cpp
+++ b/libs/binder/tests/binderRpcTest.cpp
@@ -614,25 +614,20 @@
                     status = session->setupPreconnectedClient({}, [=]() {
                         return connectTo(UnixSocketAddress(addr.c_str()));
                     });
-                    if (status == OK) goto success;
                     break;
                 case SocketType::UNIX:
                     status = session->setupUnixDomainClient(addr.c_str());
-                    if (status == OK) goto success;
                     break;
                 case SocketType::VSOCK:
                     status = session->setupVsockClient(VMADDR_CID_LOCAL, vsockPort);
-                    if (status == OK) goto success;
                     break;
                 case SocketType::INET:
                     status = session->setupInetClient("127.0.0.1", serverInfo.port);
-                    if (status == OK) goto success;
                     break;
                 default:
                     LOG_ALWAYS_FATAL("Unknown socket type");
             }
-            LOG_ALWAYS_FATAL("Could not connect %s", statusToString(status).c_str());
-        success:
+            CHECK_EQ(status, OK) << "Could not connect: " << statusToString(status);
             ret.sessions.push_back({session, session->getRootObject()});
         }
         return ret;