From f295dd421172ecca4ee6ea708bd080e9f440e7a2 Mon Sep 17 00:00:00 2001 From: huangdarwin Date: Fri, 17 Feb 2023 19:08:04 +0000 Subject: [PATCH] HDR: Add colorinfo to exception logs. Format.toString unfortunately doesn't log colorInfo, and as Format holds a very large set of values, it's unclear that it should. ColorInfo is useful for codec exceptions though, so log this in ExportException.createForCodec. PiperOrigin-RevId: 510475520 --- .../main/java/androidx/media3/transformer/ExportException.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/ExportException.java b/libraries/transformer/src/main/java/androidx/media3/transformer/ExportException.java index 9e63c28d3b..7d87b6b418 100644 --- a/libraries/transformer/src/main/java/androidx/media3/transformer/ExportException.java +++ b/libraries/transformer/src/main/java/androidx/media3/transformer/ExportException.java @@ -231,6 +231,9 @@ public final class ExportException extends Exception { boolean isDecoder, Format format) { String details = "format=" + format; + if (isVideo) { + details += ", colorInfo=" + format.colorInfo; + } return createForCodec(cause, errorCode, isVideo, isDecoder, details); }