sjs/media
1
0
Fork 0
mirror of https://github.com/samsonjs/media.git synced 2026-03-29 10:05:48 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove old TODOs and reassign another TODO.

Browse source 
* The AdvancedFrameProcessor calls use() in updateProgramAndDraw().
* The AdvancedFrameProcessor has the same input and output dimensions.

PiperOrigin-RevId: 437231350
This commit is contained in:
hschlueter 2022-03-25 13:22:22 +00:00 committed by Ian Baker
parent 9c2ee5a224
commit 9cd37385d0
3 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
library/common/src/main/java/com/google/android/exoplayer2/util/GlProgram.java
View file

@ -146,8 +146,6 @@ public final class GlProgram {
* <p>Call this in the rendering loop to switch between different programs.
*/
public void use() {
// TODO(b/214975934): When multiple GL programs are supported by Transformer, make sure
// to call use() to switch between programs.
GLES20.glUseProgram(programId);
GlUtil.checkGlError();
}

2
library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/AdvancedFrameProcessorPixelTest.java
View file

@ -58,8 +58,6 @@ public final class AdvancedFrameProcessorPixelTest {
private @MonotonicNonNull GlFrameProcessor advancedFrameProcessor;
private int inputTexId;
private int outputTexId;
// TODO(b/214975934): Once the frame processors are allowed to have different input and output
// dimensions, get the output dimensions from the frame processor.
private int width;
private int height;

2
library/transformer/src/main/java/com/google/android/exoplayer2/transformer/GlFrameProcessor.java
View file

@ -32,7 +32,7 @@ import java.io.IOException;
* </ol>
*/
public interface GlFrameProcessor {
// TODO(b/214975934): Investigate whether all configuration can be moved to initialize by
// TODO(b/213313666): Investigate whether all configuration can be moved to initialize by
// using a placeholder surface until the encoder surface is known. If so, convert
// configureOutputSize to a simple getter.