From 3d0f43a149c846670480c993e09def1a2d2eeeea Mon Sep 17 00:00:00 2001 From: ibaker Date: Mon, 24 Oct 2022 16:51:08 +0000 Subject: [PATCH] Use int for an unsigned byte value in Cea608Decoder This is a no-op, but it's more 'correct' because it avoids any potential sign mix-ups that come from storing an unsigned byte (with a potentially set MSB) in a signed java byte variable. PiperOrigin-RevId: 483409798 --- .../com/google/android/exoplayer2/text/cea/Cea608Decoder.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/library/extractor/src/main/java/com/google/android/exoplayer2/text/cea/Cea608Decoder.java b/library/extractor/src/main/java/com/google/android/exoplayer2/text/cea/Cea608Decoder.java index 0d50c3f709..5eb95468c3 100644 --- a/library/extractor/src/main/java/com/google/android/exoplayer2/text/cea/Cea608Decoder.java +++ b/library/extractor/src/main/java/com/google/android/exoplayer2/text/cea/Cea608Decoder.java @@ -458,8 +458,8 @@ public final class Cea608Decoder extends CeaDecoder { ccData.reset(subtitleData.array(), subtitleData.limit()); boolean captionDataProcessed = false; while (ccData.bytesLeft() >= packetLength) { - byte ccHeader = - packetLength == 2 ? CC_IMPLICIT_DATA_HEADER : (byte) ccData.readUnsignedByte(); + int ccHeader = packetLength == 2 ? CC_IMPLICIT_DATA_HEADER : ccData.readUnsignedByte(); + int ccByte1 = ccData.readUnsignedByte(); int ccByte2 = ccData.readUnsignedByte();