commit 8cfd2971ecdceb8ceb57a5332829a7d2d47f8b5a
author Treehugger Robot <treehugger-gerrit@google.com> Wed May 17 21:30:18 2017 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed May 17 21:30:20 2017 +0000
tree 2376714c4a6ba77dee1a808a102e490d4899c56c
parent 7ce0f00c0857d11b80818e5172d52dc41ea1bcdc
parent eb45db2cf4fc73103e21546441fcc53303983ab2

Merge "Revert "logd: remove start filtration from flushTo""

logd/LogBuffer.cpp

diff --git a/logd/LogBuffer.cpp b/logd/LogBuffer.cpp
index fbed83b..ded6c8c 100644
--- a/logd/LogBuffer.cpp
+++ b/logd/LogBuffer.cpp
@@ -1142,6 +1142,10 @@
             continue;
         }
 
+        if (element->getRealTime() <= start) {
+            continue;
+        }
+
         // NB: calling out to another object with wrlock() held (safe)
         if (filter) {
             int ret = (*filter)(element, arg);
@@ -1168,10 +1172,11 @@
         unlock();
 
         // range locking in LastLogTimes looks after us
-        log_time next = element->flushTo(reader, this, privileged, sameTid);
+        max = element->flushTo(reader, this, privileged, sameTid);
 
-        if (next == element->FLUSH_ERROR) return next;
-        if (next > max) max = next;
+        if (max == element->FLUSH_ERROR) {
+            return max;
+        }
 
         skip = maxSkip;
         rdlock();