Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-21 Thread Aman Gupta
On Sun, May 20, 2018 at 12:55 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Sun, May 20, 2018 at 11:37:22AM -0700, Aman Gupta wrote: > > On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > > > > > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: > > > > On Fri, May 18, 2018 at 11

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Michael Niedermayer
On Sun, May 20, 2018 at 11:37:22AM -0700, Aman Gupta wrote: > On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > > > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: > > > On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: > > >> From: Aman Gupta > > >> > > >> These fields will allow

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread James Almer
On 5/20/2018 3:37 PM, Aman Gupta wrote: > On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > >> On 5/19/2018 6:31 PM, Michael Niedermayer wrote: >>> On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: From: Aman Gupta These fields will allow the mpegts demuxer to expose

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Aman Gupta
On Sat, May 19, 2018 at 2:56 PM James Almer wrote: > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: > > On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: > >> From: Aman Gupta > >> > >> These fields will allow the mpegts demuxer to expose details about > >> the PMT/program which crea

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Jan Ekström
On Sun, May 20, 2018 at 2:06 PM, Jan Ekström wrote: > On Sun, May 20, 2018 at 1:54 PM, Carl Eugen Hoyos wrote: >> >> Reading the original submission above, isn't the whole point of the >> patch to add public symbols to help downstream with real-world >> issues? >> >> Carl Eugen > > Yes, currently

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Jan Ekström
On Sun, May 20, 2018 at 1:54 PM, Carl Eugen Hoyos wrote: > > Reading the original submission above, isn't the whole point of the > patch to add public symbols to help downstream with real-world > issues? > > Carl Eugen Yes, currently the API user has no visibility over how the programs are being

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-20 Thread Carl Eugen Hoyos
2018-05-19 23:48 GMT+02:00, James Almer : > On 5/19/2018 6:31 PM, Michael Niedermayer wrote: >> On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: >>> From: Aman Gupta >>> >>> These fields will allow the mpegts demuxer to expose details about >>> the PMT/program which created the AVProgra

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-19 Thread James Almer
On 5/19/2018 6:31 PM, Michael Niedermayer wrote: > On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: >> From: Aman Gupta >> >> These fields will allow the mpegts demuxer to expose details about >> the PMT/program which created the AVProgram and its AVStreams. >> >> In mpegts, a PMT which

Re: [FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-19 Thread Michael Niedermayer
On Fri, May 18, 2018 at 11:15:04AM -0700, Aman Gupta wrote: > From: Aman Gupta > > These fields will allow the mpegts demuxer to expose details about > the PMT/program which created the AVProgram and its AVStreams. > > In mpegts, a PMT which advertises streams has a version number > which can be

[FFmpeg-devel] [PATCH v4 1/3] avformat: add fields to AVProgram/AVStream for PMT change tracking

2018-05-18 Thread Aman Gupta
From: Aman Gupta These fields will allow the mpegts demuxer to expose details about the PMT/program which created the AVProgram and its AVStreams. In mpegts, a PMT which advertises streams has a version number which can be incremented at any time. When the version changes, the pids which corresp