Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-27 Thread Marc-Antoine ARNAUD
The command line option already exists: the field_order indicate if it's top/top or Bottom/Bottom order. But to set a correct patch for that, I need to update: - set the top_field_first at the opening encoder - in mpeg2 encoder: check the frame property on the filed_order, and if it's unknown take

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-27 Thread Vittorio Giovara
On Mon, Oct 27, 2014 at 4:13 PM, Marc-Antoine ARNAUD wrote: > Hi Vittorio, > > Yeah I'm agreed with your's patches to setup field_order parameter context. > But after doing that, it require to setup this information from the > field_order to he output stream. > And it's the reason of the patch to

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-27 Thread Marc-Antoine ARNAUD
Hi Vittorio, Yeah I'm agreed with your's patches to setup field_order parameter context. But after doing that, it require to setup this information from the field_order to he output stream. And it's the reason of the patch to set the interlaced mode (TT or BB) in the Mpeg2 codec header correctly.

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-27 Thread Vittorio Giovara
On Mon, Oct 27, 2014 at 9:50 AM, Marc-Antoine ARNAUD wrote: > Someone can give a feedback on this patch ? > Thanks. > > Marc-Antoine If I get the problem correctly, the codec should take the interlaced state of the container and correctly mark its frames as such. Unfortunately this problem is not

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-27 Thread Luca Barbato
On 22/10/14 18:17, Marc-Antoine ARNAUD wrote: Hi Hendrik, Sorry for the long time ... In fact the property can be set using options "field_order", and can be set in the stream during the encoding. This patch will provide that functionality because nothing connect the avconv option to the encoder

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-27 Thread Marc-Antoine ARNAUD
Someone can give a feedback on this patch ? Thanks. Marc-Antoine ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-10-22 Thread Marc-Antoine ARNAUD
Hi Hendrik, Sorry for the long time ... In fact the property can be set using options "field_order", and can be set in the stream during the encoding. This patch will provide that functionality because nothing connect the avconv option to the encoder. May be the code can be correct during a simpl

Re: [libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-07-23 Thread Hendrik Leppkes
Am 23.07.2014 17:16 schrieb "Marc-Antoine Arnaud" < arnaud.marcanto...@gmail.com>: > > --- > libavcodec/mpeg12enc.c | 2 +- > libavcodec/mpegvideo_enc.c | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c > index f87286a..84

[libav-devel] [PATCH] set top_field_first and use it to write gop header

2014-07-23 Thread Marc-Antoine Arnaud
--- libavcodec/mpeg12enc.c | 2 +- libavcodec/mpegvideo_enc.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c index f87286a..84d9508 100644 --- a/libavcodec/mpeg12enc.c +++ b/libavcodec/mpeg12enc.c @@ -424,7 +424,7 @@ void ff