Commit 28bd4661 by ojw28 Committed by GitHub

Merge pull request #3371 from andymiao858/dev-v2

Fix FLV AVCVIDEOPACKET  CTS Type Issue
parents 24cca807 69e50a43
......@@ -79,7 +79,8 @@ import com.google.android.exoplayer2.video.AvcConfig;
@Override
protected void parsePayload(ParsableByteArray data, long timeUs) throws ParserException {
int packetType = data.readUnsignedByte();
int compositionTimeMs = data.readUnsignedInt24();
int compositionTimeMs = data.readSignedInt24();
timeUs += compositionTimeMs * 1000L;
// Parse avc sequence header in case this was not done before.
if (packetType == AVC_PACKET_TYPE_SEQUENCE_HEADER && !hasOutputFormat) {
......
......@@ -257,6 +257,14 @@ public final class ParsableByteArray {
}
/**
* Reads the next three bytes as an signed value.
*/
public int readSignedInt24() {
int ui24 = readUnsignedInt24();
return (ui24 & 0x800000L) >>> 23 == 1 ? (ui24 | 0xff000000) : ui24;
}
/**
* Reads the next three bytes as a signed value in little endian order.
*/
public int readLittleEndianInt24() {
......
......@@ -335,6 +335,22 @@ public final class ParsableByteArrayTest {
}
@Test
public void testReadPositiveSignedInt24() {
byte[] data = { 0x01, 0x02, (byte) 0xFF };
ParsableByteArray byteArray = new ParsableByteArray(data);
assertThat(byteArray.readSignedInt24()).isEqualTo(0x0102FF);
assertThat(byteArray.getPosition()).isEqualTo(3);
}
@Test
public void testReadNegativeSignedInt24() {
byte[] data = { (byte)0xFF, 0x02, (byte) 0x01 };
ParsableByteArray byteArray = new ParsableByteArray(data);
assertThat(byteArray.readSignedInt24()).isEqualTo(0xFFFF0201);
assertThat(byteArray.getPosition()).isEqualTo(3);
}
@Test
public void testReadLittleEndianUnsignedShort() {
ParsableByteArray byteArray = new ParsableByteArray(new byte[] {
0x01, (byte) 0xFF, 0x02, (byte) 0xFF
......
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