From 3bc0172188d938c626da99559b13f59251668da0 Mon Sep 17 00:00:00 2001 From: huangdarwin Date: Thu, 20 Apr 2023 09:27:49 +0100 Subject: [PATCH] Remove extra param from createForVideoFrameProcessingException This arg was always the same value, so it can be simplified. PiperOrigin-RevId: 525677735 --- .../java/androidx/media3/transformer/ExportException.java | 8 +++++--- .../androidx/media3/transformer/VideoSamplePipeline.java | 6 ++---- 2 files changed, 7 insertions(+), 7 deletions(-) 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 897bb2ea53..5d5d32b6b1 100644 --- a/libraries/transformer/src/main/java/androidx/media3/transformer/ExportException.java +++ b/libraries/transformer/src/main/java/androidx/media3/transformer/ExportException.java @@ -284,12 +284,14 @@ public final class ExportException extends Exception { * Creates an instance for a {@link VideoFrameProcessor} related exception. * * @param cause The cause of the failure. - * @param errorCode See {@link #errorCode}. * @return The created instance. */ /* package */ static ExportException createForVideoFrameProcessingException( - VideoFrameProcessingException cause, int errorCode) { - return new ExportException("Video frame processing error", cause, errorCode); + VideoFrameProcessingException cause) { + return new ExportException( + "Video frame processing error", + cause, + ExportException.ERROR_CODE_VIDEO_FRAME_PROCESSING_FAILED); } /** diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/VideoSamplePipeline.java b/libraries/transformer/src/main/java/androidx/media3/transformer/VideoSamplePipeline.java index b28438878c..7842a5a0c2 100644 --- a/libraries/transformer/src/main/java/androidx/media3/transformer/VideoSamplePipeline.java +++ b/libraries/transformer/src/main/java/androidx/media3/transformer/VideoSamplePipeline.java @@ -164,8 +164,7 @@ import org.checkerframework.dataflow.qual.Pure; @Override public void onError(VideoFrameProcessingException exception) { errorConsumer.accept( - ExportException.createForVideoFrameProcessingException( - exception, ExportException.ERROR_CODE_VIDEO_FRAME_PROCESSING_FAILED)); + ExportException.createForVideoFrameProcessingException(exception)); } @Override @@ -180,8 +179,7 @@ import org.checkerframework.dataflow.qual.Pure; } }); } catch (VideoFrameProcessingException e) { - throw ExportException.createForVideoFrameProcessingException( - e, ExportException.ERROR_CODE_VIDEO_FRAME_PROCESSING_FAILED); + throw ExportException.createForVideoFrameProcessingException(e); } }