From e17b3d7f8384dd32ec5515c127ca5f1c3d3c28bc Mon Sep 17 00:00:00 2001
From: samrobinson <samrobinson@google.com>
Date: Tue, 14 May 2024 04:43:20 -0700
Subject: [PATCH] Omit "last: []" from debug trace json if less than 10 event
 logged.

Reduces the boilerplate from the json produced without any information
loss.

PiperOrigin-RevId: 633540269
---
 .../java/androidx/media3/effect/DebugTraceUtil.java  | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/libraries/effect/src/main/java/androidx/media3/effect/DebugTraceUtil.java b/libraries/effect/src/main/java/androidx/media3/effect/DebugTraceUtil.java
index 6ef451bc72..4ea3b83ebd 100644
--- a/libraries/effect/src/main/java/androidx/media3/effect/DebugTraceUtil.java
+++ b/libraries/effect/src/main/java/androidx/media3/effect/DebugTraceUtil.java
@@ -459,11 +459,15 @@ public final class DebugTraceUtil {
       for (EventLog eventLog : firstLogs) {
         jsonWriter.value(eventLog.toString());
       }
-      jsonWriter.endArray().name("last").beginArray();
-      for (EventLog eventLog : lastLogs) {
-        jsonWriter.value(eventLog.toString());
+      jsonWriter.endArray();
+      if (!lastLogs.isEmpty()) {
+        jsonWriter.name("last").beginArray();
+        for (EventLog eventLog : lastLogs) {
+          jsonWriter.value(eventLog.toString());
+        }
+        jsonWriter.endArray();
       }
-      jsonWriter.endArray().endObject();
+      jsonWriter.endObject();
     }
   }
 }