Re: [libav-devel] [PATCH] lavc: Deprecate rtp_callback field

2015-10-28 Thread Vittorio Giovara
On Wed, Oct 28, 2015 at 4:05 PM, Luca Barbato <lu_z...@gentoo.org> wrote: >> >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> >> --- >> libavcodec/avcodec.h | 6 ++ >> libavcodec/mpegvideo_enc.c | 8 >> libavcodec

[libav-devel] [PATCH] lavc: Deprecate rtp_callback field

2015-10-28 Thread Vittorio Giovara
be). So, for the reasons stated above, and being used by only a single encoder (mpegvideo), and theoretically available only through a single muxer (rtp), this field is deemed unnecessary, overcomplicated, and not belonging to libavcodec. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.

Re: [libav-devel] [PATCH 04/11] yadif: update frame rate

2015-10-28 Thread Vittorio Giovara
On Wed, Oct 28, 2015 at 5:48 PM, John Stebbins wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > (cherry picked from ffmpeg commit 31619584556466e4beab98e9b04ed4c5ba0db178) > --- > libavfilter/vf_yadif.c | 4

Re: [libav-devel] [PATCH 01/12] lavfi: add a frame_rate field to AVFilterLink.

2015-10-28 Thread Vittorio Giovara
On Wed, Oct 28, 2015 at 5:24 AM, John Stebbins <stebb...@jetheaddev.com> wrote: > On 10/27/2015 07:51 PM, Vittorio Giovara wrote: >> >> On Tue, Oct 27, 2015 at 9:17 PM, John Stebbins <stebb...@jetheaddev.com> >> wrote: >>> >>> From: Nicolas Geo

[libav-devel] [PATCH] png: read and write stereo3d frame side data information

2015-10-27 Thread Vittorio Giovara
From: Kirill Gavrilov <kir...@sview.ru> Use optional sTER chunk defining side-by-side stereo pair within "Extensions to the PNG 1.2 Specification", version 1.3.0. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Minor nits, applies to libav, simiplifies re

Re: [libav-devel] [PATCH] wrapped_avframe: Use a friendlier name

2015-10-27 Thread Vittorio Giovara
On Tue, Oct 27, 2015 at 6:01 PM, Luca Barbato wrote: > The end user reading 'wrapped_avframe' would be otherwise puzzled. > > Reported-By: Daemon404 > --- > libavcodec/wrapped_avframe.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [libav-devel] [PATCH 08/12] avfilter/vf_framepack: Check and update frame_rate

2015-10-27 Thread Vittorio Giovara
On Tue, Oct 27, 2015 at 9:17 PM, John Stebbins wrote: > From: Michael Niedermayer > > The frame_rate update was missing leaving the output frame rate > wrong. > > Signed-off-by: Michael Niedermayer > (cherry picked from commit

Re: [libav-devel] [PATCH 07/12] vf_interlace: fix frame rate

2015-10-27 Thread Vittorio Giovara
On Tue, Oct 27, 2015 at 10:15 PM, Luca Barbato wrote: > On 27/10/15 21:40, John Stebbins wrote: >> From: Michael Niedermayer >> >> Signed-off-by: Michael Niedermayer >> (cherry picked from commit 227b4458fb434875b127a0c2306fa3b899a770bf)

Re: [libav-devel] [PATCH 01/12] lavfi: add a frame_rate field to AVFilterLink.

2015-10-27 Thread Vittorio Giovara
On Tue, Oct 27, 2015 at 9:17 PM, John Stebbins wrote: > From: Nicolas George > > (cherry picked from commit 7b42036b3b23c85f473bf9369e37fa8da22eaf93) (this comment is valid for the next patches too) there is no such hash in the libav tree,

Re: [libav-devel] [PATCH] avpicture: Deprecate the single fields

2015-10-27 Thread Vittorio Giovara
On Tue, Oct 27, 2015 at 9:54 PM, Luca Barbato wrote: > Silence pointless warnings from gcc. > --- > > As suggested by courmish. It seems working decently > > libavcodec/avcodec.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/avcodec.h

Re: [libav-devel] [PATCH 1/2] h264: Print user data SEI under normal debug verbosity

2015-10-26 Thread Vittorio Giovara
On Thu, Oct 22, 2015 at 2:13 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Drop the need of setting -debug bugs since it's not a bug, and the > message is already under a AV_LOG_DEBUG log level. Instead only print > it when there is an actual string in it. ping s

Re: [libav-devel] [PATCH 6/6] avpacket: Provide an alloc and a free function for the struct

2015-10-23 Thread Vittorio Giovara
On Fri, Oct 23, 2015 at 11:11 AM, Luca Barbato wrote: > Pave the way for having the size of the AVPacket struct not part > of the ABI. > --- > doc/APIchanges| 4 > libavcodec/avcodec.h | 34 ++ > libavcodec/avpacket.c | 33

[libav-devel] [PATCH 3/3] mpegvideo_enc: Fix encoding videos with less frames than the delay of the encoder

2015-10-23 Thread Vittorio Giovara
return an empty packet, which would mean the encoder has finished, while it has not encoded any frame. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/mpegvideo_enc.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git

[libav-devel] [PATCH 2/3] mpegvideo_enc: Merge ifs with identical conditions

2015-10-23 Thread Vittorio Giovara
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavcodec/mpegvideo_enc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index 226a24f..9dd30e6 100644 ---

Re: [libav-devel] [PATCH] cmdutils: Check for and report the correct codec capability

2015-10-23 Thread Vittorio Giovara
On Mon, Oct 19, 2015 at 1:57 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > --- > cmdutils.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/cmdutils.c b/cmdutils.c > index 5cb22f6..e4bd74f 100644 > --- a/cmdutils.c > +++ b

[libav-devel] [PATCH 1/3] mpegvideo_enc: Factor new_picture unref out

2015-10-23 Thread Vittorio Giovara
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavcodec/mpegvideo_enc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index e5ff3ed..226a24f

Re: [libav-devel] [PATCH] screenpresso: Drop parameter change check

2015-10-23 Thread Vittorio Giovara
On Mon, Oct 19, 2015 at 2:34 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Size can never change, allowing us to use ff_reget_buffer, and to > simplify the code a little. > --- > libavcodec/screenpresso.c | 29 - > 1 file changed,

[libav-devel] [PATCH] h264: Change default field order for interlaced frames with missing sei

2015-10-23 Thread Vittorio Giovara
jority. Bug-Id: ffmpeg ticket #3147 Signed-off-by: Thomas Mundt <loud...@yahoo.de> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/h264.c | 6 +- 1 file changed, 5 insertions(+), 1

[libav-devel] [PATCH] avformat/mov: Autodetect mp3s which need parsing

2015-10-23 Thread Vittorio Giovara
From: Michael Niedermayer <mich...@niedermayer.cc> mp3 packets all have the same duration and number of samples if their duration indicated in the container varies then thats an indication that they are not 1 mp3 packet each. Bug-id: ffmpeg ticket #4938 / bugzilla 899 Signed-off-by: Vi

[libav-devel] [PATCH] hap: Set avctx.bits_per_coded_sample

2015-10-23 Thread Vittorio Giovara
From: Tom Butterworth <bangno...@gmail.com> Fixes an issue where alpha is ignored in some players. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/hapenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/hapenc.c b/libavcodec/hapenc.c i

Re: [libav-devel] [PATCH 3/6] avpacket: Replace av_free_packet with av_packet_unref

2015-10-23 Thread Vittorio Giovara
On Fri, Oct 23, 2015 at 11:11 AM, Luca Barbato wrote: > +2015-xx-xx - xxx - lavc 57.6.0 - avcodec.h > + Deprecate av_free_packet, av_packet_unref replaces it > + and resets the packet in a more consistent way. may i suggest a slightly different wording, so that the

Re: [libav-devel] [PATCH 5/6] avpacket: Deprecate av_dup_packet

2015-10-23 Thread Vittorio Giovara
On Fri, Oct 23, 2015 at 11:11 AM, Luca Barbato wrote: > As documented, `av_dup_packet` is broken by design, `av_packet_ref` > matches the AVFrame ref-counted API and can be safely used instead. > --- > doc/APIchanges| 2 ++ > libavcodec/avcodec.h | 5 -

Re: [libav-devel] [PATCH] h264: Change default field order for interlaced frames with missing sei

2015-10-23 Thread Vittorio Giovara
On Fri, Oct 23, 2015 at 3:38 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 23/10/15 15:24, Vittorio Giovara wrote: >> From: Thomas Mundt <loud...@yahoo.de> >> >> Default field order to top field first when interlaced frame is >> detected and pic_struct_p

Re: [libav-devel] [PATCH] avformat/mov: Autodetect mp3s which need parsing

2015-10-23 Thread Vittorio Giovara
On Fri, Oct 23, 2015 at 3:31 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 23/10/15 15:29, Vittorio Giovara wrote: >> +// If mp3 packets have a non constant duration, they could need a parser >> +if (st->codec->codec_id == AV_CODEC_ID_MP3 &&

[libav-devel] [PATCH] nuv: Cast the source data array

2015-10-23 Thread Vittorio Giovara
From: Michael Niedermayer <mich...@niedermayer.cc> Silence gcc warning 'libavcodec/nuv.c:81:19: warning: passing argument 3 of av_image_copy from incompatible pointer type' Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/nuv.c | 3 ++- 1 fil

Re: [libav-devel] [PATCH 11/11] lavc: Deprecate AVPicture structure and related functions

2015-10-22 Thread Vittorio Giovara
On Mon, Oct 19, 2015 at 2:10 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > On Wed, Oct 14, 2015 at 11:33 AM, Vittorio Giovara > <vittorio.giov...@gmail.com> wrote: >> This structure served as a bridge between data pointers and frames, but >> it su

[libav-devel] [PATCH 2/2] h264: Print the complete user message

2015-10-22 Thread Vittorio Giovara
Previously the message was cut off at 256th byte. This will allow dumping the complete x264 encode info when needed. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Amended following the suggestions received, except I didn't check for char validity since the forma

[libav-devel] [PATCH 1/2] h264: Print user data SEI under normal debug verbosity

2015-10-22 Thread Vittorio Giovara
Drop the need of setting -debug bugs since it's not a bug, and the message is already under a AV_LOG_DEBUG log level. Instead only print it when there is an actual string in it. --- libavcodec/h264_sei.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h264_sei.c

[libav-devel] [PATCH] lavc: Move deprecation warning to documentation

2015-10-22 Thread Vittorio Giovara
attribute_deprecated triggers a heavy amount of warning in every use of the structure when Libav is compiled with gcc, rather than clang. Move the notice in the documentation section instead. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/avcodec.h | 4 +---

Re: [libav-devel] [PATCH] Add libkvazaar HEVC encoder

2015-10-22 Thread Vittorio Giovara
On Thu, Oct 22, 2015 at 6:06 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 22/10/15 15:11, Vittorio Giovara wrote: >> On Sat, Oct 10, 2015 at 2:44 PM, Luca Barbato <lu_z...@gentoo.org> wrote: >>> From: Arttu Ylä-Outinen <arttu.yla-outi...@tut.fi> >

Re: [libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-22 Thread Vittorio Giovara
On Mon, Oct 19, 2015 at 2:23 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> > --- > Removed the unneded midstream parameter change. > Vittorio > > configure | 1 + &g

Re: [libav-devel] [PATCH] Add libkvazaar HEVC encoder

2015-10-22 Thread Vittorio Giovara
On Sat, Oct 10, 2015 at 2:44 PM, Luca Barbato wrote: > From: Arttu Ylä-Outinen > > Signed-off-by: Arttu Ylä-Outinen > Signed-off-by: Luca Barbato > --- > > Arttu kindly updated it, as an experimental

Re: [libav-devel] [PATCH] img2enc: Make sure the images are atomically written

2015-10-22 Thread Vittorio Giovara
On Tue, Oct 20, 2015 at 7:50 PM, Luca Barbato wrote: > --- > libavformat/img2enc.c | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c > index eeb19c0..d5b6646 100644 > --- a/libavformat/img2enc.c > +++

Re: [libav-devel] [PATCH] libx264: Make sure the extradata are padded

2015-10-22 Thread Vittorio Giovara
On Tue, Oct 20, 2015 at 9:13 PM, Luca Barbato wrote: > --- > libavcodec/libx264.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c > index 50de007..c5c888d 100644 > --- a/libavcodec/libx264.c > +++

Re: [libav-devel] [PATCH] dvenc: Validate the frame size before copying it

2015-10-22 Thread Vittorio Giovara
On Sun, Oct 18, 2015 at 11:46 PM, Luca Barbato wrote: > --- > > Would make sense to support this kind of files? (see Bug 879 to see something > that > at least now looks like that) > > libavformat/dvenc.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git

[libav-devel] [PATCH] cmdutils: Check for the correct codec capability

2015-10-19 Thread Vittorio Giovara
--- cmdutils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmdutils.c b/cmdutils.c index 5cb22f6..8df5dbf 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -996,7 +996,7 @@ static void print_codec(const AVCodec *c) printf("chconf "); if (c->capabilities &

Re: [libav-devel] [PATCH 2/2] cmdutils: Print general codec capabilities

2015-10-19 Thread Vittorio Giovara
On Thu, Oct 15, 2015 at 9:57 AM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2015-10-14 15:07:10) >> --- >> cmdutils.c | 29 + >> 1 file changed, 29 insertions(+) >> >> diff --git a/cmdutils.c b/cmdu

[libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-19 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Now with correct reget usage. Vittorio configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/

[libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-19 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Removed the unneded midstream parameter change. Vittorio configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c

[libav-devel] [PATCH] screenpresso: Drop parameter change check

2015-10-19 Thread Vittorio Giovara
Size can never change, allowing us to use ff_reget_buffer, and to simplify the code a little. --- libavcodec/screenpresso.c | 29 - 1 file changed, 12 insertions(+), 17 deletions(-) diff --git a/libavcodec/screenpresso.c b/libavcodec/screenpresso.c index

Re: [libav-devel] [PATCH 11/11] lavc: Deprecate AVPicture structure and related functions

2015-10-19 Thread Vittorio Giovara
On Wed, Oct 14, 2015 at 11:33 AM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > This structure served as a bridge between data pointers and frames, but > it suffers from several limitations: > - it is not refcounted and data must be copied to every time > - it cannot b

[libav-devel] [PATCH] cmdutils: Check for and report the correct codec capability

2015-10-19 Thread Vittorio Giovara
--- cmdutils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmdutils.c b/cmdutils.c index 5cb22f6..e4bd74f 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -995,8 +995,8 @@ static void print_codec(const AVCodec *c) if (c->capabilities & AV_CODEC_CAP_CHANNEL_CONF)

Re: [libav-devel] [PATCH] h264: Print the complete user message

2015-10-19 Thread Vittorio Giovara
On Fri, Oct 16, 2015 at 7:39 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2015-10-16 17:28:24) >> Previously the message was cut off at 256th byte. >> >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> >

Re: [libav-devel] [PATCH] mpeg12: Unbreak building stale code

2015-10-16 Thread Vittorio Giovara
On Fri, Oct 16, 2015 at 8:46 AM, Luca Barbato wrote: > Broken in 2d59159508c5c1830cc5da907a9454e229077320 > --- > > > libavcodec/mpeg12dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c > index

[libav-devel] [PATCH] h264: Print the complete user message

2015-10-16 Thread Vittorio Giovara
Previously the message was cut off at 256th byte. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Now without the invalid write of 1. Vittorio libavcodec/h264_sei.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/libavcodec/h264_s

Re: [libav-devel] [PATCH] wrapped_avframe: Drop a now-unused variable

2015-10-16 Thread Vittorio Giovara
On Fri, Oct 16, 2015 at 2:46 PM, Luca Barbato wrote: > --- > libavcodec/wrapped_avframe.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavcodec/wrapped_avframe.c b/libavcodec/wrapped_avframe.c > index 9d504db..e1273e4 100644 > --- a/libavcodec/wrapped_avframe.c >

[libav-devel] [PATCH] h264: Print the complete user message

2015-10-16 Thread Vittorio Giovara
Previously the message was cut off at 256th byte. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/h264_sei.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c index ddf1b6f..9f694fa

[libav-devel] [PATCH] h264: Print the complete user message

2015-10-16 Thread Vittorio Giovara
Previously the message was cut off at 256th byte. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/h264_sei.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c index ddf1b6f..b7b66e6

Re: [libav-devel] [PATCH 7/7] avcodec: Deprecate av_dup_packet

2015-10-15 Thread Vittorio Giovara
On Thu, Oct 15, 2015 at 2:26 AM, Luca Barbato wrote: > It is broken by design, use `av_packet_ref` to keep additional > references to an AVPacket. > --- > libavcodec/avcodec.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavcodec/avcodec.h

Re: [libav-devel] [PATCH 1/7] avpacket: Deprecate av_free_packet

2015-10-15 Thread Vittorio Giovara
On Thu, Oct 15, 2015 at 2:14 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 15/10/15 13:38, Vittorio Giovara wrote: >> On Thu, Oct 15, 2015 at 2:26 AM, Luca Barbato <lu_z...@gentoo.org> wrote: >>> `av_packet_unref` replaces it. >> >> Might be a tad b

Re: [libav-devel] [PATCH 1/7] avpacket: Deprecate av_free_packet

2015-10-15 Thread Vittorio Giovara
On Thu, Oct 15, 2015 at 2:26 AM, Luca Barbato wrote: > `av_packet_unref` replaces it. Might be a tad bit more verbose here, deprecating is never fun for users so each step has to be very well explained. > --- > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index

Re: [libav-devel] [PATCH 1/7] avpacket: Deprecate av_free_packet

2015-10-15 Thread Vittorio Giovara
On Thu, Oct 15, 2015 at 2:22 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > On Thu, Oct 15, 2015 at 2:14 PM, Luca Barbato <lu_z...@gentoo.org> wrote: >> On 15/10/15 13:38, Vittorio Giovara wrote: >>> On Thu, Oct 15, 2015 at 2:26 AM, Luca Barb

Re: [libav-devel] [PATCH 2/7] avconv: Replace av_dup_packet with av_packet_ref

2015-10-15 Thread Vittorio Giovara
On Thu, Oct 15, 2015 at 2:26 AM, Luca Barbato wrote: > --- > avconv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/avconv.c b/avconv.c > index 79569f6..34f3d42 100644 > --- a/avconv.c > +++ b/avconv.c > @@ -2125,7 +2125,7 @@ static void

[libav-devel] [PATCH] h264: Print the complete user message

2015-10-15 Thread Vittorio Giovara
Previously the message was cut off at 256th byte. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/h264_sei.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c index ddf1b6f..6426193

[libav-devel] [PATCH 11/11] lavc: Deprecate AVPicture structure and related functions

2015-10-14 Thread Vittorio Giovara
and linesize arrays directly. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/avcodec.h| 50 + libavcodec/avpicture.c | 3 ++- libavcodec/imgconvert.c | 2 ++ 3 files changed, 25 insertions(+), 30 deletions(-)

[libav-devel] [PATCH 07/11] Replace any remaining AVPicture function with imgutils

2015-10-14 Thread Vittorio Giovara
avpicture_get_size -> av_image_get_buffer_size Use the imgutils function directly, no actual need for a wrapper. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/pamenc.c | 8 +--- libavcodec/pnm_parser.c | 4 +++- libavcodec/pnmenc.c

[libav-devel] [PATCH 10/11] utils: Use data buffers directly instead of AVPicture

2015-10-14 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/utils.c | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index fa18530..63f09ff 100644 --- a/libavcodec/utils.c +++ b/libavcodec/u

[libav-devel] [PATCH 08/11] lavc: Add data and linesize to AVSubtitleRect

2015-10-14 Thread Vittorio Giovara
to deprecate AVSubtitleRect.pict. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- avplay.c | 10 - doc/APIchanges | 4 libavcodec/avcodec.h | 13 +-- libavcodec/dvbsub.c| 6 +++--- libavcodec/dvbsubdec.

[libav-devel] [PATCH 09/11] avplay: Replace the AVPicture functions with imgutils

2015-10-14 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- avplay.c | 66 1 file changed, 25 insertions(+), 41 deletions(-) diff --git a/avplay.c b/avplay.c index b666eb7..dd2073c 100644 --- a/avplay.c +++ b/av

[libav-devel] [PATCH 04/11] roqvideodec: Replace the AVPicture functions with imgutils

2015-10-14 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/roqvideodec.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavcodec/roqvideodec.c b/libavcodec/roqvideodec.c index 0ce6943..acdae75 100644 --- a/libavcodec/roqvideodec.c +++ b/liba

[libav-devel] [PATCH 01/11] msrle: Use AVFrame instead of AVPicture

2015-10-14 Thread Vittorio Giovara
Callers always use a frame and cast it to AVPicture, change ff_msrle_decode() to accept an AVFrame directly instead. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/aasc.c | 2 +- libavcodec/bmp.c | 2 +- libavcodec/msrle.c| 2 +- libavcodec/msrl

[libav-devel] [PATCH 05/11] dpx: Replace the AVPicture functions with imgutils

2015-10-14 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/dpxenc.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/libavcodec/dpxenc.c b/libavcodec/dpxenc.c index 17db94a..adcb529 100644 --- a/libavcodec/dpxenc.c +++ b/libavcodec/dp

[libav-devel] [PATCH 02/11] rawenc: Replace the AVPicture functions with imgutils

2015-10-14 Thread Vittorio Giovara
From: Luca Barbato <lu_z...@gentoo.org> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/rawenc.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavcodec/rawenc.c b/libavcodec/rawenc.c index cc55b3a..60bd0c7 100644 ---

[libav-devel] [PATCH 06/11] nuv: Replace the AVPicture functions with imgutils

2015-10-14 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/nuv.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libavcodec/nuv.c b/libavcodec/nuv.c index c56003c..9db85fd 100644 --- a/libavcodec/nuv.c +++ b/libavcodec/nuv.c @@ -74,9 +74,12 @@

[libav-devel] [PATCH 03/11] rawdec: Replace the AVPicture functions with imgutils

2015-10-14 Thread Vittorio Giovara
From: Luca Barbato <lu_z...@gentoo.org> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/rawdec.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c index 3b69e49..28

Re: [libav-devel] [PATCH 01/11] msrle: Use AVFrame instead of AVPicture

2015-10-14 Thread Vittorio Giovara
On Wed, Oct 14, 2015 at 11:57 AM, wm4 <nfx...@googlemail.com> wrote: > On Wed, 14 Oct 2015 11:33:18 +0200 > Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > >>... > > Patches 01-07,09,10 just do simple replacements of AVPicture usage (in > particular avp

[libav-devel] [PATCH 2/2] cmdutils: Print general codec capabilities

2015-10-14 Thread Vittorio Giovara
--- cmdutils.c | 29 + 1 file changed, 29 insertions(+) diff --git a/cmdutils.c b/cmdutils.c index 065ba49..5cb22f6 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -977,6 +977,35 @@ static void print_codec(const AVCodec *c) printf("%s %s [%s]:\n", encoder ? "Encoder"

[libav-devel] [PATCH 1/2] cmdutils: Add auto to threading capabilities report

2015-10-14 Thread Vittorio Giovara
And use 'none' when all capabilities are missing. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- cmdutils.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/cmdutils.c b/cmdutils.c index a4a4ae0..065ba49 100644 --- a/cmdutils.c +++ b/cmdu

[libav-devel] [PATCH] lavc: AV-prefix a few left out capabalities

2015-10-14 Thread Vittorio Giovara
--- libavcodec/libx264.c | 2 +- libavcodec/mpeg12dec.c| 2 +- libavcodec/qsvdec_h2645.c | 2 +- libavcodec/qsvdec_mpeg2.c | 2 +- libavcodec/qsvenc_hevc.c | 2 +- libavcodec/qsvenc_mpeg2.c | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/libx264.c

[libav-devel] [PATCH] fate: Move screenpress to the appropriate screen capture file

2015-10-14 Thread Vittorio Giovara
--- tests/fate/screen.mak | 3 +++ tests/fate/video.mak | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/fate/screen.mak b/tests/fate/screen.mak index 26e6736..0536909 100644 --- a/tests/fate/screen.mak +++ b/tests/fate/screen.mak @@ -41,6 +41,9 @@ fate-g2m4: CMD =

[libav-devel] [PATCH] APIchanges: Fill in missing dates and hashes

2015-10-14 Thread Vittorio Giovara
--- doc/APIchanges | 58 +- 1 file changed, 29 insertions(+), 29 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 26f8816..8d3106e 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -17,71 +17,71 @@ API changes, most recent

Re: [libav-devel] [PATCH] mimic: drop AVPicture usage

2015-10-13 Thread Vittorio Giovara
On Tue, Oct 13, 2015 at 2:48 PM, wm4 wrote: > Work on the AVFrame references directly. > > Instead of setting up a flipped/swapped "view" on the pictures, > flip/swap them when returning decoded frames to the API user. > --- > libavcodec/mimic.c | 31

Re: [libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-12 Thread Vittorio Giovara
On Fri, Oct 9, 2015 at 7:44 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2015-10-09 14:14:08) >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> >> --- >> +/* Handle midstream parameter change */ >> +if (ctx

[libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-12 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Updated with Anton's review. Vittorio configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/

Re: [libav-devel] [PATCH] avformat: Do not use AVFMT_RAWPICTURE

2015-10-12 Thread Vittorio Giovara
On Mon, Oct 12, 2015 at 4:24 PM, Luca Barbato wrote: > There are no formats supporting it anymore and it is deprecated. > Update the documentation accordingly. > --- > diff --git a/libavformat/avformat.h b/libavformat/avformat.h > index aa11cff..5caf73a 100644 > ---

Re: [libav-devel] [PATCH] avconv: Reindent

2015-10-12 Thread Vittorio Giovara
On Mon, Oct 12, 2015 at 4:26 PM, Luca Barbato wrote: > --- > avconv.c | 53 + > 1 file changed, 25 insertions(+), 28 deletions(-) > > To be squashed with the previous patch maybe. > ok with me (squashed or not, maybe yes)

[libav-devel] [PATCH] libschroedinger: Properly use AVFrame API

2015-10-12 Thread Vittorio Giovara
Rather than copying data buffers around, allocate a proper frame, and use the standard AVFrame functions. This effectively makes the decoder capable of direct rendering. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/libschroedinger.c

[libav-devel] [PATCH] qtrle: Properly use AVFrame API

2015-10-12 Thread Vittorio Giovara
Rather than copying data buffers around, just add a reference to the current frame. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/qtrleenc.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/libavcodec/qtrleenc.c b/liba

[libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-09 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/rscc.c

Re: [libav-devel] [PATCH 1/3] wrapped_avframe: Initial implementation

2015-10-09 Thread Vittorio Giovara
Please don't use "initial implementation", it's either done or not done ;) On Fri, Oct 9, 2015 at 4:26 PM, Luca Barbato wrote: > --- > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/avcodec.h | 1 + > libavcodec/codec_desc.c

[libav-devel] [PATCH] innoHeim/Rsupport Screen Capture Codec decoder

2015-10-08 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/rscc.c

Re: [libav-devel] [PATCH] dict: Change return type of av_dict_copy()

2015-10-08 Thread Vittorio Giovara
On Sun, Oct 4, 2015 at 1:27 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > av_dict_set() could return an error, so forward it appropriately. > > Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> > --- > doc/APIchanges | 4

Re: [libav-devel] [PATCH] avconv: Do not try to configure filter outputs without streams

2015-10-07 Thread Vittorio Giovara
On Mon, Oct 5, 2015 at 8:50 PM, Luca Barbato wrote: > Prevent a NULL-dereference. > > CC: libav-sta...@libav.org > --- > avconv_filter.c | 8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/avconv_filter.c b/avconv_filter.c > index 976d521..0127580

[libav-devel] closing down ABI breaking time

2015-10-06 Thread Vittorio Giovara
Hi all, it's been about two months since the big major bump, and ABI hunt season has been quite intense. Is there anything left that people would like to address before we close down changing ABI and move to release/12? -- Vittorio ___ libav-devel

[libav-devel] [PATCH] g2m: Relax resolution change constraints

2015-10-06 Thread Vittorio Giovara
Do not fail when original resolution is smaller than current one, as the frame buffer is resized automatically. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/g2meet.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/g2me

[libav-devel] [PATCH] dict: Change return type of av_dict_copy()

2015-10-04 Thread Vittorio Giovara
av_dict_set() could return an error, so forward it appropriately. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- doc/APIchanges | 4 libavutil/dict.c| 11 --- libavutil/dict.h| 4 +++- libavutil/version.h | 2 +- 4 files changed, 16 insertions

Re: [libav-devel] [PATCH][RFC] avpicture: Fix type correctness

2015-10-03 Thread Vittorio Giovara
On Sat, Oct 3, 2015 at 1:47 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2015-10-01 00:06:30) >> This silence a few warnings and matches more closely av_image_*: >> - avpicture_fill() reads a const pointer >> - avpicture_layout() uses uint8

Re: [libav-devel] [PATCH][RFC] avpicture: Fix type correctness

2015-10-03 Thread Vittorio Giovara
On Sat, Oct 3, 2015 at 2:26 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2015-10-03 14:22:31) >> On Sat, Oct 3, 2015 at 1:47 PM, Anton Khirnov <an...@khirnov.net> wrote: >> > Quoting Vittorio Giovara (2015-10-01 00:06:30) >> >&g

[libav-devel] [PATCH] h263: Drop uninitialized variable in log message

2015-10-02 Thread Vittorio Giovara
--- libavcodec/h263dec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index 3a0f4f7..fd1e57b 100644 --- a/libavcodec/h263dec.c +++ b/libavcodec/h263dec.c @@ -236,8 +236,8 @@ static int decode_slice(MpegEncContext *s)

Re: [libav-devel] [PATCH] dcadec: reorganise context data

2015-10-02 Thread Vittorio Giovara
On Fri, Oct 2, 2015 at 1:48 PM, Alexandra Hájková wrote: > place primary audio coding header data into DCAAudioHeader > structure and channel related data to DCAChan structure > --- > libavcodec/dca.h| 70 - > libavcodec/dcadec.c | 290 >

[libav-devel] [PATCH 07/10] eac3dec: Always initialize gain array

2015-10-02 Thread Vittorio Giovara
Bug-id: CID 609303 --- libavcodec/eac3dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/eac3dec.c b/libavcodec/eac3dec.c index b9d079c..2302da7 100644 --- a/libavcodec/eac3dec.c +++ b/libavcodec/eac3dec.c @@ -200,7 +200,7 @@ void

[libav-devel] [PATCH 02/10] dvbsubdec: Fix function return type

2015-10-02 Thread Vittorio Giovara
--- libavcodec/dvbsubdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/dvbsubdec.c b/libavcodec/dvbsubdec.c index d2b95bc..f1d02d7 100644 --- a/libavcodec/dvbsubdec.c +++ b/libavcodec/dvbsubdec.c @@ -1175,7 +1175,7 @@ static int

[libav-devel] [PATCH 10/10] hevc: Simplify logical check

2015-10-02 Thread Vittorio Giovara
The intended meaning "if this block is the first block in a slice then its left boundary is a slice boundary". Silence a gcc warning. --- libavcodec/hevc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index a802e1e..07b7de1 100644 ---

[libav-devel] [PATCH 03/10] h264: Use the correct argument context in debug log

2015-10-02 Thread Vittorio Giovara
--- libavcodec/h264_cabac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h264_cabac.c b/libavcodec/h264_cabac.c index f4e451d..2b847ed 100644 --- a/libavcodec/h264_cabac.c +++ b/libavcodec/h264_cabac.c @@ -2072,7 +2072,7 @@ decode_intra_mb:

[libav-devel] [PATCH 05/10] h263dec: Initialize variable before use in log function

2015-10-02 Thread Vittorio Giovara
--- libavcodec/h263dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index 3a0f4f7..ee2a4b0 100644 --- a/libavcodec/h263dec.c +++ b/libavcodec/h263dec.c @@ -236,9 +236,9 @@ static int decode_slice(MpegEncContext *s)

[libav-devel] [PATCH 01/10] dv: Mark internal frame reference as const

2015-10-02 Thread Vittorio Giovara
Fix warning due to frame assignment in dvenc. All uses of the reference in dvdec are read only, except the ones in the main decoding function, so use the frame pointer directly there. --- libavcodec/dv.h| 2 +- libavcodec/dvdec.c | 13 +++-- 2 files changed, 8 insertions(+), 7

[libav-devel] [PATCH 09/10] dict: Use av_freep() for key and value variables

2015-10-02 Thread Vittorio Giovara
Fix a warning from clang incompatible-pointer-types-discards-qualifiers. --- libavutil/dict.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavutil/dict.c b/libavutil/dict.c index 7213bf2..6a8a06e 100644 --- a/libavutil/dict.c +++ b/libavutil/dict.c @@ -80,8 +80,10

[libav-devel] [PATCH 08/10] ffv1enc: Always initialize alpha value

2015-10-02 Thread Vittorio Giovara
Avoid storing uninitialized data, even if not read later. --- libavcodec/ffv1enc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/ffv1enc.c b/libavcodec/ffv1enc.c index 32b3711..31a387c 100644 --- a/libavcodec/ffv1enc.c +++ b/libavcodec/ffv1enc.c @@ -318,7 +318,7

[libav-devel] [PATCH 04/10] rangecoder: Use AV_RB16 instead of bytestream_get_be16

2015-10-02 Thread Vittorio Giovara
Fix a rather lenghty warning from clang: rangecoder.c:58:34: warning: passing 'uint8_t **' (aka 'unsigned char **') to parameter of type 'const uint8_t **' (aka 'const unsigned char **') discards qualifiers in nested pointer types

[libav-devel] [PATCH 06/10] dcadec: Always initialize return variable

2015-10-02 Thread Vittorio Giovara
Fix a lenghty warning on clang. --- libavcodec/dcadec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c index 47d3c44..3365828 100644 --- a/libavcodec/dcadec.c +++ b/libavcodec/dcadec.c @@ -1147,7 +1147,7 @@ static float

<    11   12   13   14   15   16   17   18   19   20   >