mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-12-24 17:20:36 +00:00
Code Issues 1 Projects Releases Wiki Activity

h264parse: fix some failures in testcases

Browse source 
Move logic of judgement whether or not to insert AU Delimiter before
draining.
Besides, aud_needed flag is reset to TRUE when skip.

https://bugzilla.gnome.org/show_bug.cgi?id=736213
This commit is contained in:
Hyunjun Ko 2017-03-08 17:09:28 +09:00 committed by Sebastian Dröge
parent 0968bad8a4
commit a997a99fdd
1 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
gst/videoparsers/gsth264parse.c
View file

@ -1199,6 +1199,7 @@ gst_h264_parse_handle_frame (GstBaseParse * parse,
if (current_off == 0) {
GST_DEBUG_OBJECT (h264parse, "skipping broken nal");
*skipsize = nalu.offset;
h264parse->aud_needed = TRUE;
goto skip;
} else {
GST_DEBUG_OBJECT (h264parse, "terminating au");
@ -1232,20 +1233,25 @@ gst_h264_parse_handle_frame (GstBaseParse * parse,
"broken/invalid nal Type: %d %s, Size: %u will be dropped",
nalu.type, _nal_name (nalu.type), nalu.size);
*skipsize = nalu.size;
h264parse->aud_needed = TRUE;
goto skip;
}
/* Judge whether or not to insert AU Delimiter in case of byte-stream
* If we're in the middle of au, we don't need to insert aud.
* Otherwise, we honor the result in gst_h264_parse_process_nal.
* Note that this should be done until draining if it's happening.
*/
if (h264parse->align == GST_H264_PARSE_ALIGN_NAL && !h264parse->aud_needed)
h264parse->aud_insert = FALSE;
if (nonext)
break;
/* if no next nal, we know it's complete here */
au_complete = gst_h264_parse_collect_nal (h264parse, data, size, &nalu);
/* Judge whether or not to insert AU Delimiter in case of byte-stream */
if (h264parse->align == GST_H264_PARSE_ALIGN_NAL) {
if (!h264parse->aud_needed)
h264parse->aud_insert = FALSE;
h264parse->aud_needed = au_complete;
break;
}