MPEG-TS the ongoing story

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

MPEG-TS the ongoing story

Russel Winder
Now delving into the C code of MPEG-TS in GStreamer and I am wondering why
there is no:

gst_mpegts_section_from_eit

along with all associated code and tests, even though there are:

gst_mpegts_section_from_nit
gst_mpegts_section_from_sdt
gst_mpegts_section_from_pat
gst_mpegts_section_from_pmt

I am wondering if I am missing something other than "it's never been needed
before".

--
Russel.
===========================================
Dr Russel Winder      t: +44 20 7585 2200
41 Buckmaster Road    m: +44 7770 465 077
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: MPEG-TS the ongoing story

Sebastian Dröge-3
On Wed, 2019-05-22 at 10:32 +0100, Russel Winder wrote:
>
> I am wondering if I am missing something other than "it's never been
> needed before".

That seems to be the case, yes. Please send a patch if you need it :)

--
Sebastian Dröge, Centricular Ltd · https://www.centricular.com


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

signature.asc (981 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: MPEG-TS the ongoing story

Russel Winder
On Wed, 2019-05-22 at 13:20 +0300, Sebastian Dröge wrote:
> On Wed, 2019-05-22 at 10:32 +0100, Russel Winder wrote:
> > I am wondering if I am missing something other than "it's never been
> > needed before".
>
> That seems to be the case, yes. Please send a patch if you need it :)

I am guessing then that I need to do the C thing for entry into the MPEG-TS
library itself rather than just fill the gap in Rust in the MPEG-TS wrapper.
And I promised myself never to do any C programming again. Oh well.

Would changes get into GStreamer 1.16.1 or would it have to wait for 1.18.0?

--
Russel.
===========================================
Dr Russel Winder      t: +44 20 7585 2200
41 Buckmaster Road    m: +44 7770 465 077
London SW11 1EN, UK   w: www.russel.org.uk


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: MPEG-TS the ongoing story

Sebastian Dröge-3
On Wed, 2019-05-22 at 15:15 +0100, Russel Winder wrote:

> On Wed, 2019-05-22 at 13:20 +0300, Sebastian Dröge wrote:
> > On Wed, 2019-05-22 at 10:32 +0100, Russel Winder wrote:
> > > I am wondering if I am missing something other than "it's never been
> > > needed before".
> >
> > That seems to be the case, yes. Please send a patch if you need it :)
>
> I am guessing then that I need to do the C thing for entry into the MPEG-TS
> library itself rather than just fill the gap in Rust in the MPEG-TS wrapper.
> And I promised myself never to do any C programming again. Oh well.
That can't sometimes be prevented unfortunately, unless you want to
rewrite the whole MPEG-TS library :)

> Would changes get into GStreamer 1.16.1 or would it have to wait for 1.18.0?

Probably has to wait until 1.18, but that can be discussed once the
change is there and we see how intrusive/risky it is and how important
it is to have it backported.

--
Sebastian Dröge, Centricular Ltd · https://www.centricular.com


_______________________________________________
gstreamer-devel mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel

signature.asc (981 bytes) Download Attachment