Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegts: Factorize version checking code out

2015-04-30 Thread Michael Niedermayer
On Wed, Apr 29, 2015 at 10:24:09PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/mpegts.c | 22 ++ > 1 file changed, 14 insertions(+), 8 deletions(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegts: Factorize version checking code out

2015-04-29 Thread JULIAN GARDNER
Its nice to see this added, but why 4 years ago when I pushed these same changes, was I told it was not being added because it would break older streams created with ffmpeg. Has there been a change in policy now to try to adhere to specifications. Just wondering boys as my personal build has had

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegts: Factorize version checking code out

2015-04-29 Thread Michael Niedermayer
On Wed, Apr 29, 2015 at 10:30:56PM +0200, Hendrik Leppkes wrote: > On Wed, Apr 29, 2015 at 10:24 PM, Michael Niedermayer > wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/mpegts.c | 22 ++ > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > d

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegts: Factorize version checking code out

2015-04-29 Thread Hendrik Leppkes
On Wed, Apr 29, 2015 at 10:24 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/mpegts.c | 22 ++ > 1 file changed, 14 insertions(+), 8 deletions(-) > > diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c > index 0e5c2ba..d707cc3 1

[FFmpeg-devel] [PATCH 1/2] avformat/mpegts: Factorize version checking code out

2015-04-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mpegts.c | 22 ++ 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 0e5c2ba..d707cc3 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -580,6 +580,