diff options
author | Reimar Döffinger <Reimar.Doeffinger@gmx.de> | 2008-04-20 23:54:40 +0000 |
---|---|---|
committer | Reimar Döffinger <Reimar.Doeffinger@gmx.de> | 2008-04-20 23:54:40 +0000 |
commit | b189e6705ab47937248877a620be1a047d9741fe (patch) | |
tree | c269181ccf77a36400bac328f56062ce82d045b8 | |
parent | 58720ebd9e671f654d89d8e401d0b4dffb79122b (diff) | |
download | ffmpeg-b189e6705ab47937248877a620be1a047d9741fe.tar.gz |
Simplify ff_split_xiph_headers
Originally committed as revision 12914 to svn://svn.ffmpeg.org/ffmpeg/trunk
-rw-r--r-- | libavcodec/xiph.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/libavcodec/xiph.c b/libavcodec/xiph.c index fc197b6bb6..6554264648 100644 --- a/libavcodec/xiph.c +++ b/libavcodec/xiph.c @@ -24,7 +24,7 @@ int ff_split_xiph_headers(uint8_t *extradata, int extradata_size, int first_header_size, uint8_t *header_start[3], int header_len[3]) { - int i, j; + int i; if (extradata_size >= 6 && AV_RB16(extradata) == first_header_size) { int overall_len = 6; @@ -39,20 +39,19 @@ int ff_split_xiph_headers(uint8_t *extradata, int extradata_size, } } else if (extradata_size >= 3 && extradata_size < INT_MAX - 0x1ff && extradata[0] == 2) { int overall_len = 3; - for (i=0,j=1; i<2; i++,j++) { + extradata++; + for (i=0; i<2; i++, extradata++) { header_len[i] = 0; - for (; overall_len < extradata_size && extradata[j]==0xff; j++) { + for (; overall_len < extradata_size && *extradata==0xff; extradata++) { header_len[i] += 0xff; overall_len += 0xff + 1; } - overall_len += extradata[j]; + header_len[i] += *extradata; + overall_len += *extradata; if (overall_len > extradata_size) return -1; - - header_len[i] += extradata[j]; } - header_len[2] = extradata_size - header_len[0] - header_len[1] - j; - extradata += j; + header_len[2] = extradata_size - overall_len; header_start[0] = extradata; header_start[1] = header_start[0] + header_len[0]; header_start[2] = header_start[1] + header_len[1]; |