From 62d25b47ae55351d15ee5156b81980eba08b81fe Mon Sep 17 00:00:00 2001 From: Brad Leege Date: Thu, 7 Apr 2016 11:58:04 -0500 Subject: [PATCH] [android] #4629 - Moving event queue clearing to finally block to better protect against HTTP errors --- .../com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java index f1bd5fb16..9262d95f7 100644 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/telemetry/MapboxEventManager.java @@ -648,12 +648,13 @@ public class MapboxEventManager { Response response = client.newCall(request).execute(); Log.d(TAG, "response code = " + response.code() + " for events " + events.size()); - // Reset Events - // ============ - events.removeAllElements(); } catch (Exception e) { Log.e(TAG, "FlushTheEventsTask borked: " + e); e.printStackTrace(); + } finally { + // Reset Events + // ============ + events.removeAllElements(); } return null;