From e7b8ac382e468d547b78f58eaf2dc80ba2bb6080 Mon Sep 17 00:00:00 2001
From: Yu-Han Yang <yuhany@google.com>
Date: Fri, 5 Nov 2021 15:18:55 -0700
Subject: [PATCH] Return an empty batch when there is no location to return

Bug: 205200767
Test: atest LocationManagerFineTest#testRequestFlush_Gnss
Change-Id: I98841f1e83e39289f0e09517d572041ecb058d07
---
 gnss/aidl/default/GnssBatching.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/gnss/aidl/default/GnssBatching.cpp b/gnss/aidl/default/GnssBatching.cpp
index 2444edc63b..b8be5e5baf 100644
--- a/gnss/aidl/default/GnssBatching.cpp
+++ b/gnss/aidl/default/GnssBatching.cpp
@@ -78,9 +78,6 @@ ndk::ScopedAStatus GnssBatching::start(int64_t periodNanos, int flags) {
 
 ndk::ScopedAStatus GnssBatching::flush() {
     ALOGD("flush");
-    if (mBatchedLocations.empty()) {
-        return ndk::ScopedAStatus::ok();
-    }
     std::vector<GnssLocation> copy = std::vector<GnssLocation>(mBatchedLocations);
     ndk::ScopedAStatus status;
     if (sCallback != nullptr) {