Commit 358ea75e authored by Janne Grunau's avatar Janne Grunau

Revert "h264: skip start code search if the size of the nal unit is known"

This reverts commit 87eebb34.
parent 7b3894be
...@@ -137,10 +137,7 @@ int ff_h264_check_intra_pred_mode(H264Context *h, int mode){ ...@@ -137,10 +137,7 @@ int ff_h264_check_intra_pred_mode(H264Context *h, int mode){
return mode; return mode;
} }
const uint8_t *ff_h264_decode_nal(H264Context *h, const uint8_t *src, const uint8_t *ff_h264_decode_nal(H264Context *h, const uint8_t *src, int *dst_length, int *consumed, int length){
int *dst_length, int *consumed, int length,
int nalsize_known)
{
int i, si, di; int i, si, di;
uint8_t *dst; uint8_t *dst;
int bufidx; int bufidx;
...@@ -151,9 +148,6 @@ const uint8_t *ff_h264_decode_nal(H264Context *h, const uint8_t *src, ...@@ -151,9 +148,6 @@ const uint8_t *ff_h264_decode_nal(H264Context *h, const uint8_t *src,
src++; length--; src++; length--;
if (nalsize_known) {
i = length;
} else
#if HAVE_FAST_UNALIGNED #if HAVE_FAST_UNALIGNED
# if HAVE_FAST_64BIT # if HAVE_FAST_64BIT
# define RS 7 # define RS 7
...@@ -3795,8 +3789,7 @@ static int decode_nal_units(H264Context *h, const uint8_t *buf, int buf_size){ ...@@ -3795,8 +3789,7 @@ static int decode_nal_units(H264Context *h, const uint8_t *buf, int buf_size){
hx = h->thread_context[context_count]; hx = h->thread_context[context_count];
ptr= ff_h264_decode_nal(hx, buf + buf_index, &dst_length, &consumed, ptr= ff_h264_decode_nal(hx, buf + buf_index, &dst_length, &consumed, next_avc - buf_index);
next_avc - buf_index, !!nalsize);
if (ptr==NULL || dst_length < 0){ if (ptr==NULL || dst_length < 0){
return -1; return -1;
} }
......
...@@ -610,12 +610,9 @@ int ff_h264_decode_picture_parameter_set(H264Context *h, int bit_length); ...@@ -610,12 +610,9 @@ int ff_h264_decode_picture_parameter_set(H264Context *h, int bit_length);
* @param consumed is the number of bytes used as input * @param consumed is the number of bytes used as input
* @param length is the length of the array * @param length is the length of the array
* @param dst_length is the number of decoded bytes FIXME here or a decode rbsp tailing? * @param dst_length is the number of decoded bytes FIXME here or a decode rbsp tailing?
* @param nalsize_known skip start code search if the size of the nalu is known
* @return decoded bytes, might be src+1 if no escapes * @return decoded bytes, might be src+1 if no escapes
*/ */
const uint8_t *ff_h264_decode_nal(H264Context *h, const uint8_t *src, const uint8_t *ff_h264_decode_nal(H264Context *h, const uint8_t *src, int *dst_length, int *consumed, int length);
int *dst_length, int *consumed, int length,
int nalsize_known);
/** /**
* Free any data that may have been allocated in the H264 context like SPS, PPS etc. * Free any data that may have been allocated in the H264 context like SPS, PPS etc.
......
...@@ -144,7 +144,7 @@ static inline int parse_nal_units(AVCodecParserContext *s, ...@@ -144,7 +144,7 @@ static inline int parse_nal_units(AVCodecParserContext *s,
src_length = 20; src_length = 20;
break; break;
} }
ptr= ff_h264_decode_nal(h, buf, &dst_length, &consumed, src_length, 0); ptr= ff_h264_decode_nal(h, buf, &dst_length, &consumed, src_length);
if (ptr==NULL || dst_length < 0) if (ptr==NULL || dst_length < 0)
break; break;
......
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 to comment