Commit 782f19ed by kimvde Committed by Tianyi Feng

Release SequenceAssetLoader when ended

Before, the SequenceAssetLoader was released at the end of the export.
Release resources earlier if possible.

PiperOrigin-RevId: 519952572
parent 836c49e9
......@@ -94,6 +94,7 @@ import java.util.concurrent.atomic.AtomicInteger;
private boolean audioLoopingEnded;
private boolean videoLoopingEnded;
private int processedInputsSize;
private boolean released;
private volatile long currentAssetDurationUs;
......@@ -165,7 +166,10 @@ import java.util.concurrent.atomic.AtomicInteger;
@Override
public void release() {
currentAssetLoader.release();
if (!released) {
currentAssetLoader.release();
released = true;
}
}
/**
......@@ -368,13 +372,15 @@ import java.util.concurrent.atomic.AtomicInteger;
long globalTimestampUs = totalDurationUs + inputBuffer.timeUs - currentAssetStartTimeUs;
if (isLooping && globalTimestampUs >= maxSequenceDurationUs) {
if (isMaxSequenceDurationUsFinal && !audioLoopingEnded) {
nonEndedTracks.decrementAndGet();
checkNotNull(inputBuffer.data).limit(0);
inputBuffer.setFlags(C.BUFFER_FLAG_END_OF_STREAM);
// We know that queueInputBuffer() will always return true for the underlying
// SampleConsumer so there is no need to handle the case where the sample wasn't queued.
checkState(sampleConsumer.queueInputBuffer());
audioLoopingEnded = true;
if (nonEndedTracks.decrementAndGet() == 0) {
release();
}
}
return false;
}
......@@ -387,8 +393,13 @@ import java.util.concurrent.atomic.AtomicInteger;
if (nonEndedTracks.get() == 0) {
switchAssetLoader();
}
return true;
} else {
checkState(sampleConsumer.queueInputBuffer());
if (nonEndedTracks.get() == 0) {
release();
}
}
return true;
}
checkState(sampleConsumer.queueInputBuffer());
......@@ -448,13 +459,17 @@ import java.util.concurrent.atomic.AtomicInteger;
@Override
public void signalEndOfVideoInput() {
nonEndedTracks.decrementAndGet();
boolean videoEnded =
isLooping ? videoLoopingEnded : currentMediaItemIndex == editedMediaItems.size() - 1;
if (videoEnded) {
sampleConsumer.signalEndOfVideoInput();
} else if (nonEndedTracks.get() == 0) {
switchAssetLoader();
}
if (nonEndedTracks.decrementAndGet() == 0) {
if (videoEnded) {
release();
} else {
switchAssetLoader();
}
}
}
......@@ -479,5 +494,9 @@ import java.util.concurrent.atomic.AtomicInteger;
currentAssetLoader.start();
});
}
private void release() {
handler.post(SequenceAssetLoader.this::release);
}
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment