Commit a225e887 by olly Committed by Oliver Woodman

Move @MonotonicNonNull annotations to their correct positions

PiperOrigin-RevId: 289823804
parent cea37c60
Showing with 77 additions and 78 deletions
......@@ -37,7 +37,7 @@ public final class LibflacAudioRenderer extends SimpleDecoderAudioRenderer {
private static final int NUM_BUFFERS = 16;
@MonotonicNonNull private FlacStreamMetadata streamMetadata;
private @MonotonicNonNull FlacStreamMetadata streamMetadata;
public LibflacAudioRenderer() {
this(/* eventHandler= */ null, /* eventListener= */ null);
......
......@@ -28,7 +28,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
*/
public abstract class NoSampleRenderer implements Renderer, RendererCapabilities {
@MonotonicNonNull private RendererConfiguration configuration;
private @MonotonicNonNull RendererConfiguration configuration;
private int index;
private int state;
@Nullable private SampleStream stream;
......
......@@ -141,9 +141,9 @@ public final class AmrExtractor implements Extractor {
private int numSamplesWithSameSize;
private long timeOffsetUs;
@MonotonicNonNull private ExtractorOutput extractorOutput;
@MonotonicNonNull private TrackOutput trackOutput;
@MonotonicNonNull private SeekMap seekMap;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private @MonotonicNonNull TrackOutput trackOutput;
private @MonotonicNonNull SeekMap seekMap;
private boolean hasOutputFormat;
public AmrExtractor() {
......
......@@ -101,15 +101,15 @@ public final class FlacExtractor implements Extractor {
private final SampleNumberHolder sampleNumberHolder;
@MonotonicNonNull private ExtractorOutput extractorOutput;
@MonotonicNonNull private TrackOutput trackOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private @MonotonicNonNull TrackOutput trackOutput;
private @State int state;
@Nullable private Metadata id3Metadata;
@MonotonicNonNull private FlacStreamMetadata flacStreamMetadata;
private @MonotonicNonNull FlacStreamMetadata flacStreamMetadata;
private int minFrameSize;
private int frameStartMarker;
@MonotonicNonNull private FlacBinarySearchSeeker binarySearchSeeker;
private @MonotonicNonNull FlacBinarySearchSeeker binarySearchSeeker;
private int currentFrameBytesWritten;
private long currentFrameFirstSampleNumber;
......
......@@ -74,7 +74,7 @@ public final class FlvExtractor implements Extractor {
private final ParsableByteArray tagData;
private final ScriptTagPayloadReader metadataReader;
@MonotonicNonNull private ExtractorOutput extractorOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private @States int state;
private boolean outputFirstSample;
private long mediaTagTimestampOffsetUs;
......@@ -83,8 +83,8 @@ public final class FlvExtractor implements Extractor {
private int tagDataSize;
private long tagTimestampUs;
private boolean outputSeekMap;
@MonotonicNonNull private AudioTagPayloadReader audioReader;
@MonotonicNonNull private VideoTagPayloadReader videoReader;
private @MonotonicNonNull AudioTagPayloadReader audioReader;
private @MonotonicNonNull VideoTagPayloadReader videoReader;
public FlvExtractor() {
scratch = new ParsableByteArray(4);
......
......@@ -55,7 +55,7 @@ import org.checkerframework.checker.nullness.qual.RequiresNonNull;
private final ArrayDeque<MasterElement> masterElementsStack;
private final VarintReader varintReader;
@MonotonicNonNull private EbmlProcessor processor;
private @MonotonicNonNull EbmlProcessor processor;
private @ElementState int elementState;
private int elementId;
private long elementContentSize;
......
......@@ -390,7 +390,7 @@ public class MatroskaExtractor implements Extractor {
private boolean sampleInitializationVectorRead;
// Extractor outputs.
@MonotonicNonNull private ExtractorOutput extractorOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
public MatroskaExtractor() {
this(0);
......
......@@ -112,13 +112,13 @@ public final class Mp3Extractor implements Extractor {
private final Id3Peeker id3Peeker;
// Extractor outputs.
@MonotonicNonNull private ExtractorOutput extractorOutput;
@MonotonicNonNull private TrackOutput trackOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private @MonotonicNonNull TrackOutput trackOutput;
private int synchronizedHeaderData;
@Nullable private Metadata metadata;
@MonotonicNonNull private Seeker seeker;
private @MonotonicNonNull Seeker seeker;
private boolean disableSeeking;
private long basisTimeUs;
private long samplesRead;
......
......@@ -170,7 +170,7 @@ public class FragmentedMp4Extractor implements Extractor {
private boolean processSeiNalUnitPayload;
// Extractor output.
@MonotonicNonNull private ExtractorOutput extractorOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private TrackOutput[] emsgTrackOutputs;
private TrackOutput[] cea608TrackOutputs;
......
......@@ -115,7 +115,7 @@ public final class Mp4Extractor implements Extractor, SeekMap {
private int sampleCurrentNalBytesRemaining;
// Extractor outputs.
@MonotonicNonNull private ExtractorOutput extractorOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private Mp4Track[] tracks;
private long[][] accumulatedSampleSizes;
private int firstVideoTrackIndex;
......
......@@ -27,7 +27,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
/* package */ final class TrackFragment {
/** The default values for samples from the track fragment header. */
@MonotonicNonNull public DefaultSampleValues header;
public @MonotonicNonNull DefaultSampleValues header;
/**
* The position (byte offset) of the start of fragment.
*/
......
......@@ -39,8 +39,8 @@ public class OggExtractor implements Extractor {
private static final int MAX_VERIFICATION_BYTES = 8;
@MonotonicNonNull private ExtractorOutput output;
@MonotonicNonNull private StreamReader streamReader;
private @MonotonicNonNull ExtractorOutput output;
private @MonotonicNonNull StreamReader streamReader;
private boolean streamReaderInitialized;
@Override
......
......@@ -45,9 +45,9 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
private final OggPacket oggPacket;
@MonotonicNonNull private TrackOutput trackOutput;
@MonotonicNonNull private ExtractorOutput extractorOutput;
@MonotonicNonNull private OggSeeker oggSeeker;
private @MonotonicNonNull TrackOutput trackOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private @MonotonicNonNull OggSeeker oggSeeker;
private long targetGranule;
private long payloadStartPosition;
private long currentGranule;
......
......@@ -49,7 +49,7 @@ public final class RawCcExtractor implements Extractor {
private final Format format;
private final ParsableByteArray dataScratch;
@MonotonicNonNull private TrackOutput trackOutput;
private @MonotonicNonNull TrackOutput trackOutput;
private int parserState;
private int version;
private long timestampUs;
......
......@@ -54,8 +54,8 @@ public final class Ac3Reader implements ElementaryStreamReader {
private final ParsableByteArray headerScratchBytes;
@Nullable private final String language;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
@State private int state;
private int bytesRead;
......@@ -65,7 +65,7 @@ public final class Ac3Reader implements ElementaryStreamReader {
// Used when parsing the header.
private long sampleDurationUs;
@MonotonicNonNull private Format format;
private @MonotonicNonNull Format format;
private int sampleSize;
// Used when reading the samples.
......
......@@ -50,8 +50,8 @@ public final class Ac4Reader implements ElementaryStreamReader {
private final ParsableByteArray headerScratchBytes;
@Nullable private final String language;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
@State private int state;
private int bytesRead;
......@@ -62,7 +62,7 @@ public final class Ac4Reader implements ElementaryStreamReader {
// Used when parsing the header.
private long sampleDurationUs;
@MonotonicNonNull private Format format;
private @MonotonicNonNull Format format;
private int sampleSize;
// Used when reading the samples.
......
......@@ -87,7 +87,7 @@ public final class AdtsExtractor implements Extractor {
private final ParsableByteArray scratch;
private final ParsableBitArray scratchBits;
@MonotonicNonNull private ExtractorOutput extractorOutput;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private long firstSampleTimestampUs;
private long firstFramePosition;
......
......@@ -70,9 +70,9 @@ public final class AdtsReader implements ElementaryStreamReader {
private final ParsableByteArray id3HeaderBuffer;
@Nullable private final String language;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
@MonotonicNonNull private TrackOutput id3Output;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
private @MonotonicNonNull TrackOutput id3Output;
private int state;
private int bytesRead;
......@@ -96,7 +96,7 @@ public final class AdtsReader implements ElementaryStreamReader {
// Used when reading the samples.
private long timeUs;
@MonotonicNonNull private TrackOutput currentOutput;
private @MonotonicNonNull TrackOutput currentOutput;
private long currentSampleDuration;
/**
......
......@@ -41,8 +41,8 @@ public final class DtsReader implements ElementaryStreamReader {
private final ParsableByteArray headerScratchBytes;
@Nullable private final String language;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
private int state;
private int bytesRead;
......@@ -52,7 +52,7 @@ public final class DtsReader implements ElementaryStreamReader {
// Used when parsing the header.
private long sampleDurationUs;
@MonotonicNonNull private Format format;
private @MonotonicNonNull Format format;
private int sampleSize;
// Used when reading the samples.
......
......@@ -42,8 +42,8 @@ public final class H262Reader implements ElementaryStreamReader {
private static final int START_GROUP = 0xB8;
private static final int START_USER_DATA = 0xB2;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
// Maps (frame_rate_code - 1) indices to values, as defined in ITU-T H.262 Table 6-4.
private static final double[] FRAME_RATE_VALUES = new double[] {
......
......@@ -57,9 +57,9 @@ public final class H264Reader implements ElementaryStreamReader {
private long totalBytesWritten;
private final boolean[] prefixFlags;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
@MonotonicNonNull private SampleReader sampleReader;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
private @MonotonicNonNull SampleReader sampleReader;
// State that should not be reset on seek.
private boolean hasOutputFormat;
......
......@@ -52,9 +52,9 @@ public final class H265Reader implements ElementaryStreamReader {
private final SeiReader seiReader;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private TrackOutput output;
@MonotonicNonNull private SampleReader sampleReader;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull TrackOutput output;
private @MonotonicNonNull SampleReader sampleReader;
// State that should not be reset on seek.
private boolean hasOutputFormat;
......
......@@ -38,7 +38,7 @@ public final class Id3Reader implements ElementaryStreamReader {
private final ParsableByteArray id3Header;
@MonotonicNonNull private TrackOutput output;
private @MonotonicNonNull TrackOutput output;
// State that should be reset on seek.
private boolean writingSample;
......
......@@ -51,9 +51,9 @@ public final class LatmReader implements ElementaryStreamReader {
private final ParsableBitArray sampleBitArray;
// Track output info.
@MonotonicNonNull private TrackOutput output;
@MonotonicNonNull private String formatId;
@MonotonicNonNull private Format format;
private @MonotonicNonNull TrackOutput output;
private @MonotonicNonNull String formatId;
private @MonotonicNonNull Format format;
// Parser state info.
private int state;
......
......@@ -42,8 +42,8 @@ public final class MpegAudioReader implements ElementaryStreamReader {
private final MpegAudioHeader header;
@Nullable private final String language;
@MonotonicNonNull private TrackOutput output;
@MonotonicNonNull private String formatId;
private @MonotonicNonNull TrackOutput output;
private @MonotonicNonNull String formatId;
private int state;
private int frameBytesRead;
......
......@@ -49,7 +49,7 @@ public final class PesReader implements TsPayloadReader {
private int state;
private int bytesRead;
@MonotonicNonNull private TimestampAdjuster timestampAdjuster;
private @MonotonicNonNull TimestampAdjuster timestampAdjuster;
private boolean ptsFlag;
private boolean dtsFlag;
private boolean seenFirstDts;
......
......@@ -72,7 +72,7 @@ public final class PsExtractor implements Extractor {
// Accessed only by the loading thread.
@Nullable private PsBinarySearchSeeker psBinarySearchSeeker;
@MonotonicNonNull private ExtractorOutput output;
private @MonotonicNonNull ExtractorOutput output;
private boolean hasOutputSeekMap;
public PsExtractor() {
......
......@@ -32,8 +32,8 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
*/
public final class SpliceInfoSectionReader implements SectionPayloadReader {
@MonotonicNonNull private TimestampAdjuster timestampAdjuster;
@MonotonicNonNull private TrackOutput output;
private @MonotonicNonNull TimestampAdjuster timestampAdjuster;
private @MonotonicNonNull TrackOutput output;
private boolean formatDeclared;
@Override
......
......@@ -49,9 +49,9 @@ public final class WavExtractor implements Extractor {
/** Factory for {@link WavExtractor} instances. */
public static final ExtractorsFactory FACTORY = () -> new Extractor[] {new WavExtractor()};
@MonotonicNonNull private ExtractorOutput extractorOutput;
@MonotonicNonNull private TrackOutput trackOutput;
@MonotonicNonNull private OutputWriter outputWriter;
private @MonotonicNonNull ExtractorOutput extractorOutput;
private @MonotonicNonNull TrackOutput trackOutput;
private @MonotonicNonNull OutputWriter outputWriter;
private int dataStartPosition;
private long dataEndPosition;
......
......@@ -50,7 +50,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
private final MediaCodecAsyncCallback mediaCodecAsyncCallback;
private final MediaCodec codec;
private final HandlerThread handlerThread;
@MonotonicNonNull private Handler handler;
private @MonotonicNonNull Handler handler;
private long pendingFlushCount;
private @State int state;
private Runnable codecStartRunnable;
......
......@@ -78,8 +78,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
private final ArrayDeque<MediaFormat> formats;
@GuardedBy("objectStateLock")
@MonotonicNonNull
private MediaFormat currentFormat;
private @MonotonicNonNull MediaFormat currentFormat;
@GuardedBy("objectStateLock")
private long pendingFlush;
......@@ -92,7 +91,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
private @State int state;
private final HandlerThread handlerThread;
@MonotonicNonNull private Handler handler;
private @MonotonicNonNull Handler handler;
private Runnable codecStartRunnable;
/** Creates a new instance that wraps the specified {@link MediaCodec}. */
......
......@@ -40,7 +40,7 @@ public final class SpliceInfoDecoder implements MetadataDecoder {
private final ParsableByteArray sectionData;
private final ParsableBitArray sectionHeader;
@MonotonicNonNull private TimestampAdjuster timestampAdjuster;
private @MonotonicNonNull TimestampAdjuster timestampAdjuster;
public SpliceInfoDecoder() {
sectionData = new ParsableByteArray();
......
......@@ -179,7 +179,7 @@ public abstract class DownloadService extends Service {
@StringRes private final int channelNameResourceId;
@StringRes private final int channelDescriptionResourceId;
@MonotonicNonNull private DownloadManager downloadManager;
private @MonotonicNonNull DownloadManager downloadManager;
private int lastStartId;
private boolean startedInForeground;
private boolean taskRemoved;
......
......@@ -87,7 +87,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
private final ClutDefinition defaultClutDefinition;
private final SubtitleService subtitleService;
@MonotonicNonNull private Bitmap bitmap;
private @MonotonicNonNull Bitmap bitmap;
/**
* Construct an instance for the given subtitle and ancillary page ids.
......
......@@ -57,7 +57,7 @@ public final class CacheDataSink implements DataSink {
@Nullable private OutputStream outputStream;
private long outputStreamBytesWritten;
private long dataSpecBytesWritten;
@MonotonicNonNull private ReusableBufferedOutputStream bufferedOutputStream;
private @MonotonicNonNull ReusableBufferedOutputStream bufferedOutputStream;
/**
* Thrown when IOException is encountered when writing data into sink.
......
......@@ -60,7 +60,7 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
private final DatabaseProvider databaseProvider;
@MonotonicNonNull private String tableName;
private @MonotonicNonNull String tableName;
/**
* Deletes index data for the specified cache.
......
......@@ -212,9 +212,9 @@ import org.checkerframework.checker.nullness.qual.RequiresNonNull;
private final boolean mediaSegmentEncrypted;
private final boolean initSegmentEncrypted;
@MonotonicNonNull private Extractor extractor;
private @MonotonicNonNull Extractor extractor;
private boolean isExtractorReusable;
@MonotonicNonNull private HlsSampleStreamWrapper output;
private @MonotonicNonNull HlsSampleStreamWrapper output;
// nextLoadPosition refers to the init segment if initDataLoadRequired is true.
// Otherwise, nextLoadPosition refers to the media segment.
private int nextLoadPosition;
......
......@@ -132,20 +132,20 @@ import org.checkerframework.checker.nullness.qual.RequiresNonNull;
private int[] sampleQueueTrackIds;
private Set<Integer> sampleQueueMappingDoneByType;
private SparseIntArray sampleQueueIndicesByType;
@MonotonicNonNull private TrackOutput emsgUnwrappingTrackOutput;
private @MonotonicNonNull TrackOutput emsgUnwrappingTrackOutput;
private int primarySampleQueueType;
private int primarySampleQueueIndex;
private boolean sampleQueuesBuilt;
private boolean prepared;
private int enabledTrackGroupCount;
@MonotonicNonNull private Format upstreamTrackFormat;
private @MonotonicNonNull Format upstreamTrackFormat;
@Nullable private Format downstreamTrackFormat;
private boolean released;
// Tracks are complicated in HLS. See documentation of buildTracksFromSampleStreams for details.
// Indexed by track (as exposed by this source).
@MonotonicNonNull private TrackGroupArray trackGroups;
@MonotonicNonNull private Set<TrackGroup> optionalTrackGroups;
private @MonotonicNonNull TrackGroupArray trackGroups;
private @MonotonicNonNull Set<TrackGroup> optionalTrackGroups;
// Indexed by track group.
private int @MonotonicNonNull [] trackGroupToSampleQueueIndex;
private int primaryTrackGroupIndex;
......@@ -1358,7 +1358,7 @@ import org.checkerframework.checker.nullness.qual.RequiresNonNull;
private final EventMessageDecoder emsgDecoder;
private final TrackOutput delegate;
private final Format delegateFormat;
@MonotonicNonNull private Format format;
private @MonotonicNonNull Format format;
private byte[] buffer;
private int bufferPosition;
......
......@@ -202,7 +202,7 @@ public class DefaultTimeBar extends View implements TimeBar {
private int keyCountIncrement;
private long keyTimeIncrement;
private int lastCoarseScrubXPosition;
@MonotonicNonNull private Rect lastExclusionRectangle;
private @MonotonicNonNull Rect lastExclusionRectangle;
private boolean scrubbing;
private long scrubPosition;
......
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