[Bug 736213] New: h264parse: process NAL AU DELIMITER

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

Summary: h264parse: process NAL AU DELIMITER
Classification: Platform
Product: GStreamer
Version: git
OS/Version: Linux
Status: UNCONFIRMED
Severity: enhancement
Priority: Normal
Component: gst-plugins-bad
AssignedTo: gstreamer-***@lists.freedesktop.org
ReportedBy: ***@gmail.com
QAContact: gstreamer-***@lists.freedesktop.org
GNOME version: ---


there is no reason to dump AU delimiter until we get SPS PPS as
AU is harmless NAL

for example:
AU SPS PPS IDR AU SLICE AU SLICE ..................
produce:
SPS PPS IDR AU SLICE AU SLICE ..................

iDevices don't play hls stream without AU delimiter for every frame
so If I have h264parse after encoder h264 parse dump first AU delimiter

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

--- Comment #1 from m][sko <***@gmail.com> 2014-09-06 23:34:31 UTC ---
Created an attachment (id=285598)
View: https://bugzilla.gnome.org/attachment.cgi?id=285598
Review: https://bugzilla.gnome.org/review?bug=736213&attachment=285598

process NAL AU delimiter

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

--- Comment #2 from m][sko <***@gmail.com> 2014-09-06 23:36:14 UTC ---
to 1.4 branch too plz

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

Julien Isorce <julien.isorce> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@gmail.com

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

Olivier Crete (Tester) <olivier.crete> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@ocrete.ca

--- Comment #3 from Olivier Crete (Tester) <***@ocrete.ca> 2014-09-09 14:54:27 UTC ---
Actually, we need to insert AU delimiters between AUs when outputting
byte-stream mode to create a valid stream (for MPEG-TS embedding).

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

Sebastian Dröge (slomo) <slomo> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
CC| |***@coaxion.net
Ever Confirmed|0 |1

--- Comment #4 from Sebastian Dröge (slomo) <***@coaxion.net> 2014-09-12 12:27:37 UTC ---
Yes

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

Sebastian Dröge (slomo) <slomo> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #285598|none |needs-work
status| |

--- Comment #5 from Sebastian Dröge (slomo) <***@coaxion.net> 2014-09-12 12:28:30 UTC ---
(

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

m][sko <xlazom00> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #285598|0 |1
is obsolete| |

--- Comment #6 from m][sko <***@gmail.com> 2014-09-12 23:00:14 UTC ---
Created an attachment (id=286096)
View: https://bugzilla.gnome.org/attachment.cgi?id=286096
Review: https://bugzilla.gnome.org/review?bug=736213&attachment=286096

inline au delimiter

as we discuss in other bug
https://bugzilla.gnome.org/show_bug.cgi?id=736211
I made this patch

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

m][sko <xlazom00> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #286096|0 |1
is obsolete| |

--- Comment #7 from m][sko <***@gmail.com> 2014-09-12 23:35:02 UTC ---
Created an attachment (id=286097)
View: https://bugzilla.gnome.org/attachment.cgi?id=286097
Review: https://bugzilla.gnome.org/review?bug=736213&attachment=286097

inline au delimiter

as we discuss in other bug
https://bugzilla.gnome.org/show_bug.cgi?id=736211
I made this patch

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

Olivier Crete (Tester) <olivier.crete> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #286097|none |needs-work
status| |

--- Comment #8 from Olivier Crete (Tester) <***@ocrete.ca> 2014-09-15 21:49:37 UTC ---
Review of attachment 286097:
--> (https://bugzilla.gnome.org/review?bug=736213&attachment=286097)

There shouldn't be a property for this. The AUD should always be inserted in
bytestream mode, and never in avc mode. And it should be inserted whatever the
alignment is (au or nal), the question is just if it should be in the same
GstBuffer or not.

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

--- Comment #9 from m][sko <***@gmail.com> 2014-09-16 07:01:34 UTC ---
1. ok so I will change
h264parse->align == GST_H264_PARSE_ALIGN_AU
=>
h264parse->format == GST_H264_PARSE_FORMAT_BYTE
in gst_h264_parse_pre_push_frame


2. gst_pad_push should be enaught.
do you have problem with some muxers ?
for example if you generate hls with mpegtsmux I think that this should be ok
that is reason why I flagged buffer SPS+PPS+IDR NALs with
GST_BUFFER_FLAG_DELTA_UNIT
it is maybe useless btw
GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

--- Comment #10 from m][sko <***@gmail.com> 2014-09-16 07:04:38 UTC ---
property or not
I saw some spec that AU delimiter is options

["GStreamer" (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213
GStreamer | gst-plugins-bad | git

--- Comment #11 from Sebastian Dröge (slomo) <***@coaxion.net> 2014-09-16 07:40:54 UTC ---
The AU delimiter should be in the *same* buffer for alignment=AU, otherwise
things will break. Also this probably needs some changes to also insert the AU
delimiter if in passthrough mode

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #12 from Jan Schmidt <***@noraisin.net> ---
*** Bug 724510 has been marked as a duplicate of this bug. ***

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Jan Schmidt <***@noraisin.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@angri.ru

--- Comment #13 from Jan Schmidt <***@noraisin.net> ---
*** Bug 696037 has been marked as a duplicate of this bug. ***

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #14 from Jan Schmidt <***@noraisin.net> ---
Just a note, I just pushed:

commit 48a1f2792316d01c4ea4c6baa27188ffae73c543
Author: Jan Schmidt <***@centricular.com>
Date: Fri Jul 24 02:46:21 2015 +1000

h264parse: Don't discard first AU delimiter

Don't throw away AU delimiter(s) that precede the SPS/PPS. Should
fix MPEG-TS playback on iOS/Quicktime when muxing streams that
already have AU delimiters.

See https://bugzilla.gnome.org/show_bug.cgi?id=736213 for getting
h264parse to insert AU delimiters when they don't already
exist.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Uwe Mueller <***@namez.de> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@namez.de

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
See Also| |https://bugzilla.gnome.org/
| |show_bug.cgi?id=776712

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@igalia.com

--- Comment #15 from Hyunjun Ko <***@igalia.com> ---
Hi. I'm from #Bug 776712 :)

I think if this issue can be fixed, it would be happy for all of us:)
If people don't mind, I would propose new patch based on master.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #16 from Hyunjun Ko <***@igalia.com> ---
Created attachment 346038
--> https://bugzilla.gnome.org/attachment.cgi?id=346038&action=edit
h264parse: insert AU delimiter only in case of byte-stream

Inserts AU delimeter by default if missing au delimeter from upstream.
This should be done only in case of byte-stream format.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #17 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Review of attachment 346038:
--> (https://bugzilla.gnome.org/review?bug=736213&attachment=346038)

Also, since AU delimiter is optional, there should be a property, imo.

::: gst/videoparsers/gsth264parse.c
@@ +2329,3 @@
+ gboolean ok;
+
+ gst_byte_writer_init_with_size (&bw, gst_buffer_get_size (buffer), FALSE);

Isn't there an overflow? because you are adding 48 bits to the original buffers

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #18 from Hyunjun Ko <***@igalia.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #17)

Post by "GStreamer" (GNOME Bugzilla)
Also, since AU delimiter is optional, there should be a property, imo.

Some people want that it should be by default. I have no idea of this thing.
Hope that listen to opinion of someone else. :)

Post by "GStreamer" (GNOME Bugzilla)
::: gst/videoparsers/gsth264parse.c
@@ +2329,3 @@
+ gboolean ok;
+
+ gst_byte_writer_init_with_size (&bw, gst_buffer_get_size (buffer), FALSE);
Isn't there an overflow? because you are adding 48 bits to the original
buffers

That size is not fixed as the third parameter, which means it can allocate more
if needed. But I think you're right in this case, which can be fixed at
initialization for performance.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #19 from Olivier Crête <***@ocrete.ca> ---
I think that AU delimiter should always be there in byte-stream mode as it's
required to make a valid MPEG-TS file.

As for avc1/avc3 modes, I don't think it's required, I'd be tempted to never
include it, but I have no strong feelings there.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #20 from Olivier Crête <***@ocrete.ca> ---
Review of attachment 346038:
--> (https://bugzilla.gnome.org/review?bug=736213&attachment=346038)

and you seem to be doing two copies. Maybe better just allocate a new
GstMemory, write the AUD in there and append it to the buffer.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Tim-Philipp Müller <***@zen.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------

Post by "GStreamer" (GNOME Bugzilla)
I think that AU delimiter should always be there in byte-stream
mode as it's required to make a valid MPEG-TS file.

Where is this specified? (That it is required to make a valid MPEG-TS file)

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346038|0 |1
is obsolete| |

--- Comment #22 from Hyunjun Ko <***@igalia.com> ---
Created attachment 346292
--> https://bugzilla.gnome.org/attachment.cgi?id=346292&action=edit
h264parse: insert AU delimiter only in case of byte-stream

Agree with Oliver about memory copy thing.
Patch updated

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Nicolas Dufresne (stormer) <***@ndufresne.ca> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@ndufresne.ca

--- Comment #23 from Nicolas Dufresne (stormer) <***@ndufresne.ca> ---
(In reply to Tim-Philipp Müller from comment #21)

Post by "GStreamer" (GNOME Bugzilla)

Post by "GStreamer" (GNOME Bugzilla)
I think that AU delimiter should always be there in byte-stream
mode as it's required to make a valid MPEG-TS file.

Where is this specified? (That it is required to make a valid MPEG-TS file)

I didn't have time to dig into the spec, but at least Apple requires it for
HLS:

https://developer.apple.com/library/content/documentation/NetworkingInternet/Conceptual/StreamingMediaGuide/FrequentlyAskedQuestions/FrequentlyAskedQuestions.html

I believe it's armless to have them. It's common to skip unused unit, but
uncommon to add them when missing.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #24 from Nicolas Dufresne (stormer) <***@ndufresne.ca> ---
(Forgot to mention, it's in question 10)

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #25 from Tim-Philipp Müller <***@zen.co.uk> ---
I'm quite aware that Apple requires them (see various mailing list posts that
led most people to this bug), and I also agree that it makes sense to
add/output them by default for byte-stream output.

I just don't believe what Olivier said is true, that they are required to make
a valid TS stream, but happy to be convinced otherwise :)

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #26 from Olivier Crête <***@ocrete.ca> ---
This is specified in H.222 (06/2012) on in section 2.14.1 (page 135), the first
bullet point is:

"Each AVC access unit shall contain an access unit delimiter NAL Unit;"

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@igalia.com

--- Comment #27 from Víctor Manuel Jáquez Leal <***@igalia.com> ---
Given Olivier input the patch looks right. Can we land it?

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #28 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Is that patch also ensuring the AU is there if alignment=nal instead of au?

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #29 from Hyunjun Ko <***@igalia.com> ---
(In reply to Sebastian Dröge (slomo) from comment #28)

Post by "GStreamer" (GNOME Bugzilla)
Is that patch also ensuring the AU is there if alignment=nal instead of au?

Ah, you're right.
In case of alignment=nal, this patch inserts AUD to every nal unit, which is
wrong.

I couldn't find a way to figure it out when it should insert AUD in case of
alignment=nal because a frame means a nal unit in this case, which means
we should know if this nal unit is start of au or not and also if previous nal
unit was AUD or not.

Any suggestion about this?
Or we can go this only in case of alignment=au?

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #30 from Sebastian Dröge (slomo) <***@coaxion.net> ---
(In reply to Hyunjun Ko from comment #29)

Post by "GStreamer" (GNOME Bugzilla)
Or we can go this only in case of alignment=au?

That would then still be invalid in the alignment=nal case


I think for that case it would have to be detected on the input side if the
current AU started with an AU marker, and then add it before outputting the
first NAL of the AU at the output side. Doing might need to add a little bit of
buffering.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346292|none |needs-work
status| |

--- Comment #31 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Comment on attachment 346292
--> https://bugzilla.gnome.org/attachment.cgi?id=346292
h264parse: insert AU delimiter only in case of byte-stream

Also this patch breaks some h264 streams I have here, looking

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #32 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Created attachment 347065
--> https://bugzilla.gnome.org/attachment.cgi?id=347065&action=edit
h264parse: Fix insertion of AU delimiters in alignment=au mode

We have to compensate for the new bytes added for the AU, otherwise
insertion of PPS/SPS will use wrong offsets and overwrite wrong data.

Also mark the AU delimiter blob const, and use frame->out_buffer for
storing the output to keep baseparse assumptions valid.

Still broken in alignment=nal mode.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #286097|0 |1
is obsolete| |
Attachment #286097|inline au delimiter |inline au delimiter
description| |

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Blocks| |776712

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Hyunjun Ko <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #346292|needs-work |none
status| |
Attachment #346292|0 |1
is obsolete| |

--- Comment #33 from Hyunjun Ko <***@igalia.com> ---
Created attachment 347370
--> https://bugzilla.gnome.org/attachment.cgi?id=347370&action=edit
h264parse: insert AU delimiter only in case of byte-stream

Inserts AU delimeter by default if missing au delimeter from upstream.
This should be done only in case of byte-stream format.

Note that:
We have to compensate for the new bytes added for the AU, otherwise
insertion of PPS/SPS will use wrong offsets and overwrite wrong data.

Also mark the AU delimiter blob const, and use frame->out_buffer for
storing the output to keep baseparse assumptions valid.

-----
I reprepared a patch for handling both alignment of nal and au.
I've unified slomo's patch since I changed the first proposed patch a bit.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #347065|0 |1
is obsolete| |

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #34 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Looks good to me. Now someone should check if this is also needed for h265 and
open a bug for it in that case :) Hyunjun, Olivier, do you know?

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED

--- Comment #35 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Attachment 347370 pushed as 201e71c - h264parse: insert AU delimiter only in
case of byte-stream

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #347370|none |committed
status| |

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|git master |1.11.3

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #36 from Tim-Philipp Müller <***@zen.co.uk> ---
Thanks for the reference Olivier, I stand corrected :)

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #37 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Unfortunately this breaks various tests:

$ make elements/h264parse.check
[...]
0%: Checks: 6, Failures: 6, Errors: 0

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Tim-Philipp Müller <***@zen.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |---
Severity|enhancement |blocker

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #38 from Hyunjun Ko <***@igalia.com> ---
(In reply to Sebastian Dröge (slomo) from comment #37)

Post by "GStreamer" (GNOME Bugzilla)
$ make elements/h264parse.check
[...]
0%: Checks: 6, Failures: 6, Errors: 0

Sorry for that.
I'm working on it.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #39 from Hyunjun Ko <***@igalia.com> ---
Created attachment 347445
--> https://bugzilla.gnome.org/attachment.cgi?id=347445&action=edit
h264parse: fix some failures in testcases

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

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #40 from Hyunjun Ko <***@igalia.com> ---
Created attachment 347446
--> https://bugzilla.gnome.org/attachment.cgi?id=347446&action=edit
tests: h264parse: fix failures due to insertion of au delimiter

Since insertion of aud landed, we need to change some testcases
accroding to the change.

Note that counting frames are changed in parser.c,
due to generated frames, AUD.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution|--- |FIXED

--- Comment #41 from Sebastian Dröge (slomo) <***@coaxion.net> ---
Attachment 347445 pushed as a997a99 - h264parse: fix some failures in testcases
Attachment 347446 pushed as 08219f3 - tests: h264parse: fix failures due to
insertion of au delimiter

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #347445|none |committed
status| |

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #347446|none |committed
status| |

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Víctor Manuel Jáquez Leal <***@igalia.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Blocks|776712 |

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Florent Thiéry <***@ubicast.eu> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@ubicast.eu

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Nicolas Dufresne (ndufresne) <***@ndufresne.ca> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |---

--- Comment #42 from Nicolas Dufresne (ndufresne) <***@ndufresne.ca> ---
I've been testing this today with multiple slices. With 2 slice, this code
insert an AU delimiter right in between the two slices for me, and the second
slice end up with a bumped forward timestamp.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Sebastian Dröge (slomo) <***@coaxion.net> changed:

What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|1.11.90 |1.14.4

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Guillaume Desmottes <***@gnome.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@gnome.org

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

--- Comment #43 from Tim-Philipp Müller <***@zen.co.uk> ---
Maybe we can clone this into a new bug instead of re-opening one from 18 months
ago? :)

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Tim-Philipp Müller <***@zen.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|1.14.4 |1.14.5

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Nicolas Dufresne (ndufresne) <***@ndufresne.ca> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution|--- |FIXED

--- Comment #44 from Nicolas Dufresne (ndufresne) <***@ndufresne.ca> ---
Sure, in fact let's close, I'll make one ticket with my h264/h265parse
enhancement (hopefully next week). I also implemented the same for h265.

["GStreamer" (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=736213

Tim-Philipp Müller <***@zen.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|1.14.5 |1.11.90