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/binderRpcBenchmark.cpp

diff --git a/libs/binder/tests/binderRpcBenchmark.cpp b/libs/binder/tests/binderRpcBenchmark.cpp
index e430c28..55aa57b 100644
--- a/libs/binder/tests/binderRpcBenchmark.cpp
+++ b/libs/binder/tests/binderRpcBenchmark.cpp
@@ -211,10 +211,9 @@
     for (size_t tries = 0; tries < 5; tries++) {
         usleep(10000);
         status = gSession->setupUnixDomainClient(addr.c_str());
-        if (status == OK) goto success;
+        if (status == OK) break;
     }
-    LOG(FATAL) << "Could not connect: " << statusToString(status).c_str();
-success:
+    CHECK_EQ(status, OK) << "Could not connect: " << statusToString(status).c_str();
 
     ::benchmark::RunSpecifiedBenchmarks();
     return 0;