commit 19e8de0742e84355a63feb48b7e1fc6ffbcfbae1
author Mark Salyzyn <salyzyn@google.com> Mon Aug 11 10:36:39 2014 -0700
committer Mark Salyzyn <salyzyn@google.com> Mon Sep 01 10:42:04 2014 -0700
tree 0f5844bd27171991f8ffad3c44a792d1a42726e6
parent c692410e1607a39cd6d55f4be8164fc630d9772c

logd: simplification, return NULL instead of threadexit NULL

Bug: 16822776
Change-Id: I12133226f1b48d8fdc378dea0aadd78a4aae3da8

logd/LogTimes.cpp

diff --git a/logd/LogTimes.cpp b/logd/LogTimes.cpp
index c6dc174..ea4e8c8 100644
--- a/logd/LogTimes.cpp
+++ b/logd/LogTimes.cpp
@@ -119,8 +119,7 @@
     SocketClient *client = me->mClient;
     if (!client) {
         me->error();
-        pthread_exit(NULL);
-        // NOTREACH
+        return NULL;
     }
 
     LogBuffer &logbuf = me->mReader.logbuf();
@@ -154,9 +153,6 @@
 
     unlock();
 
-    pthread_exit(NULL);
-
-    // NOTREACH
     pthread_cleanup_pop(true);
 
     return NULL;