Re: [FFmpeg-devel] [PATCH] Fixed remuxing of HDMV PGS subtitles

2015-04-11 Thread Philip Langdale
On Sun, 12 Apr 2015 00:16:45 + (UTC) Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > > > From: Niklesh Lalwani gmail.com> > > > > Fixes #2622. > > Which sample did you use for testing? I don't think we have a sample in FATE that has subtitles. I used one of my own bluray discs

Re: [FFmpeg-devel] [RFC][PATCH] avformat/flvdec: avoid reseting eof_reached to 0 silently

2015-04-11 Thread Zhang Rui
2015-04-10 22:04 GMT+08:00 wm4 : > On Fri, 10 Apr 2015 21:17:42 +0800 > Zhang Rui wrote: >> >> This kind of error handling need some more work in aviobuf.c, >> and more advises from ffmpeg developers. >> And i prefer this way than the patch I posted. > > stdio.h does it this way: FILE has an error

Re: [FFmpeg-devel] [PATCH] [FATESERVER] Support parsing version 1 headers

2015-04-11 Thread Michael Niedermayer
On Fri, Apr 10, 2015 at 11:24:20AM -0700, Timothy Gu wrote: > --- > Now with fate-recv change > > FATE.pm | 9 +++-- > fate-recv.sh | 13 ++--- > 2 files changed, 17 insertions(+), 5 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133

Re: [FFmpeg-devel] [PATCH v3] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Michael Niedermayer
On Sat, Apr 11, 2015 at 05:18:13PM -0700, Mark Reid wrote: > --- > libavformat/mxfenc.c | 102 > ++- > 1 file changed, 85 insertions(+), 17 deletions(-) applied as tomas said previously "Looks OK otherwise." thanks [...] -- Michael GnuPG fin

[FFmpeg-devel] [PATCH v3] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Mark Reid
--- libavformat/mxfenc.c | 102 ++- 1 file changed, 85 insertions(+), 17 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 7b400b3..0b8dfd6 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -313,6 +313,7 @@ ty

[FFmpeg-devel] [PATCH v3] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Mark Reid
changes since v2: * re-worded error message if incorrect audio codec used Mark Reid (1): libavformat/mxfenc: added support mxf opatom audio muxing libavformat/mxfenc.c | 102 ++- 1 file changed, 85 insertions(+), 17 deletions(-) -- 2.2.1

Re: [FFmpeg-devel] [PATCH] Fixed remuxing of HDMV PGS subtitles

2015-04-11 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > From: Niklesh Lalwani gmail.com> > > Fixes #2622. Which sample did you use for testing? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Mark Reid
On Sat, Apr 11, 2015 at 4:58 AM, Carl Eugen Hoyos wrote: > Tomas Härdin codemill.se> writes: > > > > +av_log(s, AV_LOG_ERROR, "MXF OPAtom only supports > > > 16 or 24 bits le audio\n"); > > > > This should perhaps be reworded as "Only 16- and 24-bit LE > > is implemented" since the spec

[FFmpeg-devel] [PATCH] Fixed remuxing of HDMV PGS subtitles

2015-04-11 Thread Timothy Gu
From: Niklesh Lalwani Fixes #2622. --- >From https://github.com/FFmpeg/FFmpeg/pull/125 libavformat/mpegts.h| 2 ++ libavformat/mpegtsenc.c | 3 +++ 2 files changed, 5 insertions(+) diff --git a/libavformat/mpegts.h b/libavformat/mpegts.h index 84f3098..f6a6551 100644 --- a/libavformat/mpe

Re: [FFmpeg-devel] [PATCH] tests: Add ATRAC3plus tests

2015-04-11 Thread Timothy Gu
Diego, On Fri, Apr 10, 2015 at 2:16 PM Timothy Gu wrote: > --- > > The official name is ATRAC3plus, so that is used. > > - at3p_sample1.oma is from samples.libav.org > - at3p_sample1.pcm is in incoming. > - sonateno14op27-2-cut.aa3 is a smaller version of sonateno14op27-2.aa3 in > samples.liba

Re: [FFmpeg-devel] [PATCH] Add atrac3+ tests

2015-04-11 Thread Michael Niedermayer
On Fri, Apr 10, 2015 at 01:24:57PM -0700, Timothy Gu wrote: > --- > > $ du -h * > 656K at3p_sample1.oma > 3.6M at3p_sample1.pcm > 1.9M sonateno14op27-2.aa3 > 100K sonateno14op27-2-cut.aa3 > 2.9M sonateno14op27-2-cut.pcm > > tests/fate/atrac.mak | 17 ++--- > 1 file changed, 14 i

Re: [FFmpeg-devel] [PATCH] libavformat/http.c: Make http-listen work as an input stream.

2015-04-11 Thread Michael Niedermayer
On Sat, Apr 11, 2015 at 10:34:13PM +0200, Stephan Holljes wrote: > On Sat, Apr 11, 2015 at 8:30 PM, Michael Niedermayer wrote: > > On Mon, Apr 06, 2015 at 10:25:37PM +0200, Michael Niedermayer wrote: > >> On Mon, Apr 06, 2015 at 08:29:42PM +0200, Stephan Holljes wrote: > >> [...] > >> > http.c |

Re: [FFmpeg-devel] [PATCH] libavformat/http.c: Make http-listen work as an input stream.

2015-04-11 Thread Nicolas George
Le duodi 22 germinal, an CCXXIII, Michael Niedermayer a écrit : > maybe my mail was ambigous but i only applied the first of the 2 > patches as there was a comment on the 2nd, though i can apply it too > if the mentors want As I said, the patch looks fine by me and I believe it is now Stephan's ch

Re: [FFmpeg-devel] [PATCH] libavformat/http.c: Make http-listen work as an input stream.

2015-04-11 Thread Michael Niedermayer
On Mon, Apr 06, 2015 at 10:25:37PM +0200, Michael Niedermayer wrote: > On Mon, Apr 06, 2015 at 08:29:42PM +0200, Stephan Holljes wrote: > [...] > > http.c |6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > 2df39e4d329e2a1fbdb6fa4edc49ec7381ff601f > > 0001-libavformat-http.c-F

Re: [FFmpeg-devel] [PATCH] libavformat/http.c: Make http-listen work as an input stream.

2015-04-11 Thread Nicolas George
Le septidi 17 germinal, an CCXXIII, Stephan Holljes a écrit : > From bbd8fc4e700968a977b30850b7be3ec39bad3531 Mon Sep 17 00:00:00 2001 > From: Stephan Holljes > Date: Mon, 6 Apr 2015 20:23:01 +0200 > Subject: [PATCH 2/2] libavformat/http.c: Make http-listen work as an input > stream. > > With th

Re: [FFmpeg-devel] [PATCH] avformat/rtpproto: Move dscp into context & AVOptions

2015-04-11 Thread Michael Niedermayer
On Sat, Apr 11, 2015 at 07:11:17PM +0200, Thomas Volkert wrote: > > Am 11.04.2015 um 19:00 schrieb Michael Niedermayer: > >Signed-off-by: Michael Niedermayer > >--- > > libavformat/rtpproto.c | 17 - > > 1 file changed, 8 insertions(+), 9 deletions(-) > > > >diff --git a/libavf

Re: [FFmpeg-devel] [PATCH] avformat/rtpproto: Move dscp into context & AVOptions

2015-04-11 Thread Thomas Volkert
Am 11.04.2015 um 19:00 schrieb Michael Niedermayer: Signed-off-by: Michael Niedermayer --- libavformat/rtpproto.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/libavformat/rtpproto.c b/libavformat/rtpproto.c index 0f294fb..d3e2ca0 100644 --- a/libavf

[FFmpeg-devel] [PATCH] avformat/rtpproto: Move dscp into context & AVOptions

2015-04-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/rtpproto.c | 17 - 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/libavformat/rtpproto.c b/libavformat/rtpproto.c index 0f294fb..d3e2ca0 100644 --- a/libavformat/rtpproto.c +++ b/libavformat/rtpproto.c @@ -55,6 +5

Re: [FFmpeg-devel] Refund request for Chemnitzer Linux Tage 2015

2015-04-11 Thread Thomas Volkert
Am 11.04.2015 um 03:31 schrieb compn: On Fri, 10 Apr 2015 20:58:13 +0200 Thilo Borgmann wrote: Hi! I'd like to request refunds for merchandise expenses at the Chemnitzer Linux Tage 2015, where we had manned a booth for FFmpeg. I wonder why there seems to be nothing else on the list because

Re: [FFmpeg-devel] [PATCH v2] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Carl Eugen Hoyos
Tomas Härdin codemill.se> writes: > > +av_log(s, AV_LOG_ERROR, "MXF OPAtom only supports > > 16 or 24 bits le audio\n"); > > This should perhaps be reworded as "Only 16- and 24-bit LE > is implemented" since the spec itself allows loads more. > > > +return AVERROR(

Re: [FFmpeg-devel] [PATCH v2] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Tomas Härdin
On Fri, 2015-04-10 at 20:02 -0700, Mark Reid wrote: > @@ -2055,8 +2083,35 @@ static int mxf_write_header(AVFormatContext *s) > av_log(s, AV_LOG_ERROR, "MXF D-10 only support 16 or 24 > bits le audio\n"); > } > sc->index = > ((MXFStreamContext