Commit 10d06dd1 by ibaker Committed by microkatz

Fix Dackka/Metalava errors in the effects module

Public methods may only refer to public types in their signature. This
change ensures that by switching to a public supertype everywhere.

PiperOrigin-RevId: 485568625
(cherry picked from commit 3fdfe585)
parent e48fe57c
...@@ -42,7 +42,7 @@ public interface ColorLut extends GlEffect { ...@@ -42,7 +42,7 @@ public interface ColorLut extends GlEffect {
/** This method must be executed on the same thread as other GL commands. */ /** This method must be executed on the same thread as other GL commands. */
@Override @Override
@WorkerThread @WorkerThread
default ColorLutProcessor toGlTextureProcessor(Context context, boolean useHdr) default SingleFrameGlTextureProcessor toGlTextureProcessor(Context context, boolean useHdr)
throws FrameProcessingException { throws FrameProcessingException {
return new ColorLutProcessor(context, /* colorLut= */ this, useHdr); return new ColorLutProcessor(context, /* colorLut= */ this, useHdr);
} }
......
...@@ -111,7 +111,7 @@ public class HslAdjustment implements GlEffect { ...@@ -111,7 +111,7 @@ public class HslAdjustment implements GlEffect {
} }
@Override @Override
public HslProcessor toGlTextureProcessor(Context context, boolean useHdr) public SingleFrameGlTextureProcessor toGlTextureProcessor(Context context, boolean useHdr)
throws FrameProcessingException { throws FrameProcessingException {
return new HslProcessor(context, /* hslAdjustment= */ this, useHdr); return new HslProcessor(context, /* hslAdjustment= */ this, useHdr);
} }
......
...@@ -89,7 +89,7 @@ public class RgbFilter implements RgbMatrix { ...@@ -89,7 +89,7 @@ public class RgbFilter implements RgbMatrix {
} }
@Override @Override
public MatrixTextureProcessor toGlTextureProcessor(Context context, boolean useHdr) public SingleFrameGlTextureProcessor toGlTextureProcessor(Context context, boolean useHdr)
throws FrameProcessingException { throws FrameProcessingException {
checkForConsistentHdrSetting(useHdr); checkForConsistentHdrSetting(useHdr);
return RgbMatrix.super.toGlTextureProcessor(context, useHdr); return RgbMatrix.super.toGlTextureProcessor(context, useHdr);
......
...@@ -38,7 +38,7 @@ public interface RgbMatrix extends GlEffect { ...@@ -38,7 +38,7 @@ public interface RgbMatrix extends GlEffect {
float[] getMatrix(long presentationTimeUs, boolean useHdr); float[] getMatrix(long presentationTimeUs, boolean useHdr);
@Override @Override
default MatrixTextureProcessor toGlTextureProcessor(Context context, boolean useHdr) default SingleFrameGlTextureProcessor toGlTextureProcessor(Context context, boolean useHdr)
throws FrameProcessingException { throws FrameProcessingException {
return MatrixTextureProcessor.create( return MatrixTextureProcessor.create(
context, context,
......
...@@ -147,7 +147,7 @@ public class SingleColorLut implements ColorLut { ...@@ -147,7 +147,7 @@ public class SingleColorLut implements ColorLut {
} }
@Override @Override
public ColorLutProcessor toGlTextureProcessor(Context context, boolean useHdr) public SingleFrameGlTextureProcessor toGlTextureProcessor(Context context, boolean useHdr)
throws FrameProcessingException { throws FrameProcessingException {
checkState(!useHdr, "HDR is currently not supported."); checkState(!useHdr, "HDR is currently not supported.");
......
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