Re: [libav-devel] [PATCH] vda: remove async decoder leftovers

2013-03-12 Thread Janne Grunau
On 2013-03-12 01:04:17 +0100, Luca Barbato wrote: > On 11/03/13 20:59, Anton Khirnov wrote: > > Looks ok. > > > > I assume it's tested and works. > > Surely builds, I got again my deal of issues with my macosx partition > and I couldn't complete the vlc build. I'd push this part now so at > least

[libav-devel] [PATCH 1/1] avframe: call release_buffer only if it is set

2013-03-11 Thread Janne Grunau
AVCodecContext release_buffer() shall be NULL for audio codecs using get_buffer. The backward compatibility code hence have to check before calling it. --- libavcodec/utils.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 0106b91

Re: [libav-devel] [PATCH 2/2] pthread: unref the decoded but not returned frames on close.

2013-03-11 Thread Janne Grunau
On 2013-03-11 18:07:31 +0100, Anton Khirnov wrote: > Fixes memleaks when frame mt is used and the decoder is not flushed at > the end. > --- > libavcodec/pthread.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/pthread.c b/libavcodec/pthread.c > index 280c08e..7061090 1006

Re: [libav-devel] [PATCH 1/2] h264: check that DPB is allocated before accessing it in flush_dpb()

2013-03-11 Thread Janne Grunau
On 2013-03-11 18:07:30 +0100, Anton Khirnov wrote: > --- > libavcodec/h264.c |5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/h264.c b/libavcodec/h264.c > index cad2520..95f7a8d 100644 > --- a/libavcodec/h264.c > +++ b/libavcodec/h264.c > @@ -2626,8 +262

[libav-devel] [PATCH 1/1] avframe: copy reordered_opaque in copy_props

2013-03-11 Thread Janne Grunau
Fixes video playback in applications still using reordered_opaque. --- libavutil/frame.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/frame.c b/libavutil/frame.c index 7a2a4d1..331e174 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -373,6 +373,7 @@ int av_frame_copy_pro

Re: [libav-devel] [PATCH 2/3] vf_hqdn3d: fix uninitialized variable use

2013-03-10 Thread Janne Grunau
On 2013-03-10 22:54:49 +0100, Anton Khirnov wrote: > CC:libav-sta...@libav.org > --- > libavfilter/vf_hqdn3d.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavfilter/vf_hqdn3d.c b/libavfilter/vf_hqdn3d.c > index 3251b6f..50e904b 100644 > --- a/libavfilter/vf_hqdn3d.c > +++ b/liba

Re: [libav-devel] [PATCH 1/3] vf_gradfun: fix uninitialized variable use

2013-03-10 Thread Janne Grunau
On 2013-03-10 22:54:48 +0100, Anton Khirnov wrote: > CC:libav-sta...@libav.org > --- > libavfilter/vf_gradfun.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavfilter/vf_gradfun.c b/libavfilter/vf_gradfun.c > index 05412cf..ca7ef69 100644 > --- a/libavfilter/vf_gradfun.c > +++ b/

[libav-devel] [PATCH 1/1] ra144: check buffer size before requesting a buffer

2013-03-10 Thread Janne Grunau
On 2013-03-10 22:28:49 +0100, Anton Khirnov wrote: > > On Sun, 10 Mar 2013 22:13:29 +0100, Janne Grunau > wrote: > > Avoids memory leaks on incomplete frames. > > --- > > libavcodec/ra144dec.c | 13 +++-- > > 1 file changed, 7 insertions(+), 6 d

Re: [libav-devel] [PATCH] h264: check for luma and chroma bit dept being equal

2013-03-10 Thread Janne Grunau
On 2013-03-10 02:52:14 +0100, Luca Barbato wrote: > The decoder assumes a single bit depth for all the planes while > the specification allows different bit depths for luma and chroma. > > Avoid the possible problems described in CVE-2013-2277 > --- > > Release 0.8 flavour. Janne please confirm I

[libav-devel] [PATCH 2/2] wmapro: unref skipped frames

2013-03-10 Thread Janne Grunau
--- libavcodec/wmaprodec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/wmaprodec.c b/libavcodec/wmaprodec.c index 56bb83c..7d5ac03 100644 --- a/libavcodec/wmaprodec.c +++ b/libavcodec/wmaprodec.c @@ -1388,6 +1388,7 @@ static int decode_frame(WMAProDecodeCtx *s, AVFrame *frame,

[libav-devel] [PATCH 1/2] ra144: check buffer size before requesting a buffer

2013-03-10 Thread Janne Grunau
Avoids memory leaks on incomplete frames. --- libavcodec/ra144dec.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/libavcodec/ra144dec.c b/libavcodec/ra144dec.c index e2d116c..d9aafc8 100644 --- a/libavcodec/ra144dec.c +++ b/libavcodec/ra144dec.c @@ -76,6 +76,13

Re: [libav-devel] [PATCH] lavc: use new av_frame_* functions in the old avcodec frame functions

2013-03-10 Thread Janne Grunau
On 2013-03-09 18:52:36 +0100, Anton Khirnov wrote: > --- > libavcodec/utils.c | 21 ++--- > 1 file changed, 2 insertions(+), 19 deletions(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 13efea4..9e16294 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/uti

[libav-devel] [PATCH 1/1] svq1: use av_frame_free to free refcounted frame

2013-03-10 Thread Janne Grunau
Fixes a memleak due to avcodec_free_frame not freeing the frame data. --- libavcodec/svq1dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c index c5d56ee..d9e6f7e 100644 --- a/libavcodec/svq1dec.c +++ b/libavcodec/svq1dec.c @@ -793

Re: [libav-devel] [PATCH] lavc: fix get_buffer() compatibility layer for audio.

2013-03-10 Thread Janne Grunau
On 2013-03-10 18:03:29 +0100, Anton Khirnov wrote: > planes - FF_ARRAY_ELEMS would be evaluated as unsigned and underflow > instead of being negative as was intended. > --- > libavcodec/utils.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c b/libavc

Re: [libav-devel] [PATCH 24/27] mpegvideo: reindent

2013-03-08 Thread Janne Grunau
On 2013-03-08 08:17:35 +0100, Anton Khirnov wrote: > --- > libavcodec/mpegvideo.c | 168 > > 1 file changed, 84 insertions(+), 84 deletions(-) ok assuming git diff -b is empty Janne ___ libav-devel mai

Re: [libav-devel] [PATCH 27/27] h264_refs: cosmetics, reformat

2013-03-08 Thread Janne Grunau
On 2013-03-08 08:17:38 +0100, Anton Khirnov wrote: > --- > libavcodec/h264_refs.c | 400 > ++-- > 1 file changed, 213 insertions(+), 187 deletions(-) > > diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c > index 2af47e3..b06f79d 100644 > --

Re: [libav-devel] [PATCH] atomic: Exclude the unsupported implementation headers from checkheaders

2013-03-08 Thread Janne Grunau
On 2013-03-08 15:32:53 +0200, Martin Storsjö wrote: > --- > libavutil/Makefile |4 > 1 file changed, 4 insertions(+) > > diff --git a/libavutil/Makefile b/libavutil/Makefile > index 582fc31..b6306f7 100644 > --- a/libavutil/Makefile > +++ b/libavutil/Makefile > @@ -103,6 +103,10 @@ OBJS

Re: [libav-devel] [PATCH 03/14] tty: set avg_frame_rate.

2013-03-07 Thread Janne Grunau
On 2013-03-07 10:26:09 +0100, Anton Khirnov wrote: > The container does not store any timestamps and is CFR-only. > --- > libavformat/tty.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/tty.c b/libavformat/tty.c > index f85f230..0ae1510 100644 > --- a/libavformat/tty.c >

Re: [libav-devel] [PATCH 14/14] lavc: makes sure the frame pool is in a consistent state on failure.

2013-03-07 Thread Janne Grunau
On 2013-03-07 10:26:20 +0100, Anton Khirnov wrote: > --- > libavcodec/utils.c | 21 - > 1 file changed, 16 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 770b897..376dcd1 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/uti

Re: [libav-devel] [PATCH 01/14] ffv1: fix calculating slice dimensions

2013-03-07 Thread Janne Grunau
On 2013-03-07 10:26:07 +0100, Anton Khirnov wrote: Please amend the commit msg mentioned that it was only broken for ffv1 version 2 files ant the commit which broke it: 0f13cd3187192ba0cc2b043430de6e279e7b97c3 > CC:libav-sta...@libav.org > --- > libavcodec/ffv1dec.c |4 ++-- > 1 file changed

Re: [libav-devel] [PATCH 02/14] FATE: enable multiple slices in the ffv1 vsynth test

2013-03-07 Thread Janne Grunau
On 2013-03-07 10:26:08 +0100, Anton Khirnov wrote: > This allows us to test the slice threading code. > --- > tests/fate/vcodec.mak |2 +- > tests/ref/seek/vsynth2-ffv1 | 40 > tests/ref/vsynth/vsynth1-ffv1 |4 ++-- > tests/ref/vsynth/vs

Re: [libav-devel] [PATCH 30/42] h264: convert to refcounted frames.

2013-03-07 Thread Janne Grunau
On 2013-03-07 14:28:04 +0100, Janne Grunau wrote: > On 2013-03-06 21:59:07 +0100, Janne Grunau wrote: > > On 2013-03-04 11:35:33 +0100, Anton Khirnov wrote: > > > --- > > > libavcodec/h264.c| 324 > > > +- &g

Re: [libav-devel] [PATCH 30/42] h264: convert to refcounted frames.

2013-03-07 Thread Janne Grunau
On 2013-03-06 21:59:07 +0100, Janne Grunau wrote: > On 2013-03-04 11:35:33 +0100, Anton Khirnov wrote: > > --- > > libavcodec/h264.c| 324 > > +- > > libavcodec/h264.h|7 +- > > libavcodec/h264_d

Re: [libav-devel] [PATCH] svq3: convert to refcounted frames.

2013-03-07 Thread Janne Grunau
On 2013-03-07 07:47:21 +0100, Anton Khirnov wrote: > --- > libavcodec/svq3.c | 82 > + > 1 file changed, 52 insertions(+), 30 deletions(-) ok Janne ___ libav-devel mailing list libav-devel@libav.or

Re: [libav-devel] [PATCH 08/42] AVFrame: add side data.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:11 +0100, Anton Khirnov wrote: > --- > libavutil/frame.c | 72 > + > libavutil/frame.h | 38 > 2 files changed, 110 insertions(+) lgtm Janne ___ l

Re: [libav-devel] [PATCH 07/42] Move AVFrame from lavc to lavu.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:10 +0100, Anton Khirnov wrote: > Add AVBuffer-based reference counting API to it. > --- > libavcodec/avcodec.h | 348 +--- > libavutil/Makefile |2 + > libavutil/frame.c| 401 ++ > libav

Re: [libav-devel] [PATCH 37/42] avplay: switch to new refcounted frames API

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:40 +0100, Anton Khirnov wrote: > Remove now unused cmdutils get_buffer() implementation. > --- > avplay.c | 76 +--- > cmdutils.c | 126 > > cmdutils.h | 43

Re: [libav-devel] [PATCH 36/42] avconv: convert to new refcounted AVFrame API

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:39 +0100, Anton Khirnov wrote: > --- > avconv.c | 130 > -- > avconv.h |4 +- > 2 files changed, 69 insertions(+), 65 deletions(-) lgtm Janne ___ libav-devel mailing

Re: [libav-devel] [PATCH 40/42] AVFrame: deprecate all now unused fields

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:43 +0100, Anton Khirnov wrote: > --- > libavutil/frame.h | 26 +- > 1 file changed, 25 insertions(+), 1 deletion(-) ok Janne ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailma

Re: [libav-devel] [PATCH 39/42] lavc: stop setting AVFrame.motion_subsample_log2

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:42 +0100, Anton Khirnov wrote: > It is not used inside lavc anywhere and now it makes no sense to export > it. > --- > libavcodec/error_resilience.c |1 - > libavcodec/h264.c |1 - > libavcodec/svq3.c |1 - > 3 files changed, 3 deletions(-) >

Re: [libav-devel] [PATCH 34/42] lavc decoders: properly initialize AVFrame.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:37 +0100, Anton Khirnov wrote: > --- > libavcodec/avs.c |2 ++ > libavcodec/bethsoftvideo.c |1 + > libavcodec/c93.c |3 +++ > libavcodec/cinepak.c |2 +- > libavcodec/dsicinav.c |2 +- > libavcodec/dv.c|1 + >

Re: [libav-devel] [PATCH 33/42] lavc: make up a fake frame channel layout when there is no real one.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:36 +0100, Anton Khirnov wrote: > This should ensure that a valid channel layout is always set on a frame, > until a better solution is implemented. > --- > libavcodec/utils.c | 14 -- > 1 file changed, 12 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/

Re: [libav-devel] [PATCH 30/42] h264: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:33 +0100, Anton Khirnov wrote: > --- > libavcodec/h264.c| 324 > +- > libavcodec/h264.h|7 +- > libavcodec/h264_direct.c |2 +- > libavcodec/h264_refs.c | 32 +++-- > 4 files changed, 231 insertions(+), 1

Re: [libav-devel] [PATCH 29/42] svq3: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:32 +0100, Anton Khirnov wrote: > --- > libavcodec/svq3.c | 72 > - > 1 file changed, 43 insertions(+), 29 deletions(-) > > diff --git a/libavcodec/svq3.c b/libavcodec/svq3.c > index ed8432a..cec067e 100644 > --- a/libavcod

Re: [libav-devel] [PATCH 28/42] ljpegenc: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:31 +0100, Anton Khirnov wrote: > --- > libavcodec/ljpegenc.c |5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/ljpegenc.c b/libavcodec/ljpegenc.c > index 555bffa..c7291ad 100644 > --- a/libavcodec/ljpegenc.c > +++ b/libavcodec/ljpegenc.

Re: [libav-devel] [PATCH 27/42] mpeg4videodec: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:30 +0100, Anton Khirnov wrote: > --- > libavcodec/mpeg4videodec.c | 15 +++ > 1 file changed, 11 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c > index 53da86f..5f13045 100644 > --- a/libavcodec/mpeg4videode

Re: [libav-devel] [PATCH 26/42] mpeg12: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:29 +0100, Anton Khirnov wrote: > --- > libavcodec/mpeg12.c | 31 +++ > 1 file changed, 23 insertions(+), 8 deletions(-) ok Janne ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.or

Re: [libav-devel] [PATCH 25/42] h263dec: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:28 +0100, Anton Khirnov wrote: > --- > libavcodec/h263dec.c |9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c > index 7ab9ed2..20fa2f2 100644 > --- a/libavcodec/h263dec.c > +++ b/libavcodec/h263dec.

Re: [libav-devel] [PATCH 24/42] h261dec: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:27 +0100, Anton Khirnov wrote: > --- > libavcodec/h261dec.c |3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c > index 3997172..d8ce66d 100644 > --- a/libavcodec/h261dec.c > +++ b/libavcodec/h261dec.c > @@

Re: [libav-devel] [PATCH 23/42] rv34: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:26 +0100, Anton Khirnov wrote: > --- > libavcodec/rv34.c | 31 --- > 1 file changed, 20 insertions(+), 11 deletions(-) ok Janne ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org

Re: [libav-devel] [PATCH 22/42] rv10: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:25 +0100, Anton Khirnov wrote: > --- > libavcodec/rv10.c |8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/rv10.c b/libavcodec/rv10.c > index e872d24..562e5d7 100644 > --- a/libavcodec/rv10.c > +++ b/libavcodec/rv10.c > @@ -643,7 +6

Re: [libav-devel] [PATCH 21/42] vc1dec: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:24 +0100, Anton Khirnov wrote: > --- > libavcodec/vc1dec.c | 27 +-- > 1 file changed, 13 insertions(+), 14 deletions(-) > > diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c > index 17d0e16..ea882d6 100644 > --- a/libavcodec/vc1dec.c > +++ b/lib

Re: [libav-devel] [PATCH 20/42] mpegvideo: convert to refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:23 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 24 +- > libavcodec/mpeg4videoenc.c |2 +- > libavcodec/mpegvideo.c | 511 > > libavcodec/mpegvideo.h | 59 +++-- > libavcodec/mpegvide

Re: [libav-devel] [PATCH 19/42] mpegvideo: move ref_index to Picture

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:22 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 16 > libavcodec/h263.c |8 > libavcodec/h264.c | 16 > libavcodec/h264.h |4 ++-- > libavcodec/h264_direct.c |

Re: [libav-devel] [PATCH 18/42] mpegvideo: only print dct coefficients when FF_DEBUG_DCT_COEFF is set

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:21 +0100, Anton Khirnov wrote: > Do not export them in AVFrame, since the corresponding field will be > removed. > --- > libavcodec/mpegvideo.c | 11 ++- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.

Re: [libav-devel] [PATCH 17/42] mpegvideo: move mb_type to Picture

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:20 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 42 ++-- > libavcodec/h261dec.c |6 +++--- > libavcodec/h263.c | 14 ++-- > libavcodec/h264.c | 24 ++--- >

Re: [libav-devel] [PATCH 16/42] mpegvideo: move motion_val to Picture

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:19 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 62 +- > libavcodec/h263.c | 18 +-- > libavcodec/h264.c | 16 +-- > libavcodec/h264.h |2 +- > libavcodec/h264_direct.c |8 +- > libavcodec/h26

Re: [libav-devel] [PATCH 15/42] mpegvideo: move mbskip_table to Picture

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:18 +0100, Anton Khirnov wrote: > --- > libavcodec/h263.c |2 +- > libavcodec/motion_est.c|2 +- > libavcodec/mpeg4videodec.c |8 > libavcodec/mpeg4videoenc.c |2 +- > libavcodec/mpegvideo.c |4 ++-- > libavcodec/mpegvideo.h |

Re: [libav-devel] [PATCH 14/42] mpegvideo: move qscale_table to Picture

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:17 +0100, Anton Khirnov wrote: > --- > libavcodec/h263.c|6 ++--- > libavcodec/h264.c| 20 +++ > libavcodec/h264_cabac.c |4 +-- > libavcodec/h264_cavlc.c |4 +-- > libavcodec/h264_loopfilter.c | 26 ++---

Re: [libav-devel] [PATCH 13/42] mpegvideo: change the signature of ff_print_debug_info().

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:16 +0100, Anton Khirnov wrote: > Take a Picture instead of AVFrame. > > It is needed because some fields from AVFrame printed by this function > will be moved to Picture. > --- > libavcodec/h261dec.c |2 +- > libavcodec/h263dec.c |3 ++- > libavcodec/mpeg12.c|

Re: [libav-devel] [PATCH 12/42] mpegvideo: move AVFrame.reference to Picture.reference

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:15 +0100, Anton Khirnov wrote: > --- > libavcodec/dxva2_h264.c| 10 +- > libavcodec/h264.c | 34 +- > libavcodec/h264_direct.c | 16 > libavcodec/h264_refs.c | 32 ---

Re: [libav-devel] [PATCH 11/42] mpegvideo_enc: drop support for reusing the input motion vectors.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:14 +0100, Anton Khirnov wrote: > This misfeature is most likely completely useless and conflicts with > removing the mpegvideo-specific fields from AVFrame. In the improbable > case it is actually useful, it should be reimplemented in a better way. > --- > libavcodec/motion_est

Re: [libav-devel] [PATCH 10/42] lavc decoders: work with refcounted frames.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:13 +0100, Anton Khirnov wrote: > --- > doc/multithreading.txt |5 + > libavcodec/4xm.c| 83 +++--- > libavcodec/8bps.c | 30 +- > libavcodec/8svx.c |2 +- > libavcodec/aacdec.c |3 +- > libavcodec/

Re: [libav-devel] [PATCH 03/42] AVBuffer: add a new API for buffer pools

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:06 +0100, Anton Khirnov wrote: > --- > libavutil/buffer.c | 143 > +++ > libavutil/buffer.h | 70 + > libavutil/buffer_internal.h | 32 ++ > 3 files changed, 245 insertions(+) lgtm Jan

Re: [libav-devel] [PATCH 06/42] lavc: move AVFrame.hwaccel_picture_private to Picture.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:09 +0100, Anton Khirnov wrote: > This field is private and should not be present in a public struct. It > is only used in DXVA with mpegvideo-based decoders currently. > --- > libavcodec/avcodec.h |8 > libavcodec/dxva2_h264.c |8 > libavcodec/dxv

Re: [libav-devel] [PATCH 05/42] pthread: avoid copying input packets when possible.

2013-03-06 Thread Janne Grunau
On 2013-03-04 11:35:08 +0100, Anton Khirnov wrote: > --- > libavcodec/pthread.c | 20 +--- > 1 file changed, 13 insertions(+), 7 deletions(-) lgtm Janne ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailma

Re: [libav-devel] [PATCH] lavu: add support for atomic operations.

2013-03-06 Thread Janne Grunau
On 2013-03-05 09:03:27 +0100, Anton Khirnov wrote: > > On Mon, 4 Mar 2013 15:55:50 +0100, Janne Grunau > wrote: > > On 2013-03-04 15:10:46 +0100, Anton Khirnov wrote: > > > From: "Ronald S. Bultje" > > > > > > +#else > > > + > &

Re: [libav-devel] [PATCH] lavu: add support for atomic operations.

2013-03-04 Thread Janne Grunau
On 2013-03-04 15:10:46 +0100, Anton Khirnov wrote: > From: "Ronald S. Bultje" > > These could be used for reference counting, or for keeping track of > decoding progress in references in multithreaded decoders. > > Support is provided by gcc/msvc/suncc intrinsics, with a fallback using > pthread

Re: [libav-devel] [PATCH 02/42] lavu: add a new API for reference-counted data buffers.

2013-03-04 Thread Janne Grunau
On 2013-03-04 11:35:05 +0100, Anton Khirnov wrote: > --- > libavutil/Makefile |2 + > libavutil/buffer.c | 194 ++ > libavutil/buffer.h | 195 > +++ > libavutil/buffer_internal.h | 6

Re: [libav-devel] [PATCH 02/42] lavu: add a new API for reference-counted data buffers.

2013-03-04 Thread Janne Grunau
On 2013-03-04 11:35:05 +0100, Anton Khirnov wrote: > --- > libavutil/Makefile |2 + > libavutil/buffer.c | 194 ++ > libavutil/buffer.h | 195 > +++ > libavutil/buffer_internal.h | 6

Re: [libav-devel] [PATCH 01/42] lavu: add support for atomic operations.

2013-03-04 Thread Janne Grunau
On 2013-03-04 11:35:04 +0100, Anton Khirnov wrote: > From: "Ronald S. Bultje" > > These could be used for reference counting, or for keeping track of > decoding progress in references in multithreaded decoders. > > Support is provided by gcc/msvc/suncc intrinsics, with a fallback using > pthread

Re: [libav-devel] [PATCH] h264: Different bit depth between luma and chroma is not supported

2013-03-04 Thread Janne Grunau
On 2013-03-04 11:37:27 +0100, Luca Barbato wrote: > The decoder assumes a single bitdepth for all the planes, the bit depth > specification does allow that. not clear what the specification allows: "all the planes while the specification allows different bit depths for luma and chroma." > > Av

Re: [libav-devel] [PATCH] h264: Different bitdepth betwenn luma and chroma is not supported

2013-03-04 Thread Janne Grunau
On 2013-03-04 11:26:52 +0100, Luca Barbato wrote: > The decoder assumes a single bitdepth for all the planes, the > specification does allow that. > > Avoid the possible problems pointed in CVE-2013-2277 > > CC: libav-sta...@libav.org > --- > libavcodec/h264.c | 5 + > 1 file changed, 5 inse

[libav-devel] [PATCH 1/1] fate: remove last incomplete frame from mpeg2-field-enc test

2013-03-04 Thread Janne Grunau
On 2013-03-04 08:30:23 +0100, Anton Khirnov wrote: > > On Sun, 3 Mar 2013 20:02:13 +0100, Janne Grunau > wrote: > > > > > > diff --git a/tests/ref/fate/mpeg2-field-enc > > > b/tests/ref/fate/mpeg2-field-enc > > > index 079aae4..d3ef6ba 1006

Re: [libav-devel] [PATCH 096/124] svq1dec: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-01-08 15:37:48 +0100, Anton Khirnov wrote: > --- > libavcodec/svq1dec.c | 34 +++--- > 1 file changed, 11 insertions(+), 23 deletions(-) > > diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c > index 5156042..0c3df9a 100644 > --- a/libavcodec/svq1dec.c

Re: [libav-devel] [PATCH 19/19] h264: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:19 +0100, Anton Khirnov wrote: > --- > libavcodec/h264.c| 310 > +- > libavcodec/h264.h|5 + > libavcodec/h264_direct.c |2 +- > libavcodec/h264_refs.c | 32 +++-- > 4 files changed, 225 insertions(+), 12

Re: [libav-devel] [PATCH 18/19] svq3: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:18 +0100, Anton Khirnov wrote: > --- > libavcodec/svq3.c | 51 +++ > 1 file changed, 31 insertions(+), 20 deletions(-) > > diff --git a/libavcodec/svq3.c b/libavcodec/svq3.c > index 880bda2..330f3ab 100644 > --- a/libavcodec/sv

Re: [libav-devel] [PATCH 16/19] mpeg4videodec: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:16 +0100, Anton Khirnov wrote: > --- > libavcodec/mpeg4videodec.c | 22 ++ > 1 file changed, 18 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c > index 53da86f..ed889bd 100644 > --- a/libavcodec/mpeg4

Re: [libav-devel] [PATCH 09/19] mpegvideo: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:09 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 15 +- > libavcodec/mpeg4videoenc.c |2 +- > libavcodec/mpegvideo.c | 531 > > libavcodec/mpegvideo.h | 56 +++-- > libavcodec/mpegvide

Re: [libav-devel] [PATCH 08/19] mpegvideo: move ref_index to Picture

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:08 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 16 > libavcodec/h263.c |8 > libavcodec/h264.c | 16 > libavcodec/h264.h |4 ++-- > libavcodec/h264_direct.c |

Re: [libav-devel] [PATCH 07/19] mpegvideo: only print dct coefficients when FF_DEBUG_DCT_COEFF is set

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:07 +0100, Anton Khirnov wrote: > Do not export them in AVFrame, since the corresponding field will be > removed. > --- > libavcodec/mpegvideo.c | 11 ++- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.

Re: [libav-devel] [PATCH 05/19] mpegvideo: move motion_val to Picture

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:05 +0100, Anton Khirnov wrote: > --- > libavcodec/error_resilience.c | 62 +- > libavcodec/h263.c | 18 +-- > libavcodec/h264.c | 16 +-- > libavcodec/h264.h |2 +- > libavcodec/h264_direct.c |8 +- > libavcodec/h26

Re: [libav-devel] [PATCH 03/19] mpegvideo: move qscale_table to Picture

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:03 +0100, Anton Khirnov wrote: > --- > libavcodec/h263.c|6 ++--- > libavcodec/h264.c| 20 +++ > libavcodec/h264_cabac.c |4 +-- > libavcodec/h264_cavlc.c |4 +-- > libavcodec/h264_loopfilter.c | 26 ++---

Re: [libav-devel] [PATCH 01/19] mpegvideo: move AVFrame.reference to Picture.reference

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:01 +0100, Anton Khirnov wrote: > --- > libavcodec/dxva2_h264.c| 10 +- > libavcodec/h264.c | 36 ++-- > libavcodec/h264_direct.c | 16 > libavcodec/h264_refs.c | 32 -

Re: [libav-devel] [PATCH 04/19] mpegvideo: move mbskip_table to Picture

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:51:04 +0100, Anton Khirnov wrote: > --- > libavcodec/h263.c |2 +- > libavcodec/motion_est.c|2 +- > libavcodec/mpeg4videodec.c |8 > libavcodec/mpeg4videoenc.c |2 +- > libavcodec/mpegvideo.c |4 ++-- > libavcodec/mpegvideo.h |

Re: [libav-devel] [PATCH 12/19] rv34: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-02-14 19:35:15 +0100, Luca Barbato wrote: > On 2/13/13 7:51 PM, Anton Khirnov wrote: > > @@ -1502,6 +1502,8 @@ av_cold int ff_rv34_decode_init(AVCodecContext *avctx) > > if(!intra_vlcs[0].cbppattern[0].bits) > > rv34_init_tables(); > > > > +avctx->internal->allocate_prog

Re: [libav-devel] [PATCH] pthread: convert to refcounted frames.

2013-03-03 Thread Janne Grunau
On 2013-02-13 19:50:38 +0100, Anton Khirnov wrote: > --- > doc/multithreading.txt |5 ++ > libavcodec/internal.h | 15 + > libavcodec/pthread.c | 169 > > libavcodec/thread.h| 20 -- > libavcodec/utils.c | 19 ++ >

Re: [libav-devel] [PATCH 08/15] h264: set ref_count to 0 for intra slices.

2013-02-28 Thread Janne Grunau
On 2013-02-16 21:37:47 +0100, Anton Khirnov wrote: > CC:libav-sta...@libav.org > --- > libavcodec/h264.c |5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/h264.c b/libavcodec/h264.c > index 47029a3..e187c59 100644 > --- a/libavcodec/h264.c > +++ b/libavcod

Re: [libav-devel] [PATCH 06/15] h264: do not copy ref count/ref2frm when updating per-frame context

2013-02-28 Thread Janne Grunau
On 2013-02-16 21:37:45 +0100, Anton Khirnov wrote: > They are filled in decode_slice_header() anyway. > --- > libavcodec/h264.c |2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavcodec/h264.c b/libavcodec/h264.c > index cd974d5..5749079 100644 > --- a/libavcodec/h264.c > +++ b/liba

Re: [libav-devel] [PATCH 07/15] h264: on reference overflow, reset the reference count to 0, not 1.

2013-02-28 Thread Janne Grunau
On 2013-02-16 21:37:46 +0100, Anton Khirnov wrote: > Since decode_slice_header() returns before the reference lists are > constructed, there are zero valid references. > > CC:libav-sta...@libav.org > --- > libavcodec/h264.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --gi

Re: [libav-devel] [PATCH] ffv1: convert to refcounted frames.

2013-02-19 Thread Janne Grunau
On 2013-02-19 08:25:58 +0100, Anton Khirnov wrote: > --- > libavcodec/ffv1.c|5 + > libavcodec/ffv1.h|2 ++ > libavcodec/ffv1dec.c | 47 +++ > 3 files changed, 26 insertions(+), 28 deletions(-) > > diff --git a/libavcodec/ffv1.c b/

Re: [libav-devel] [PATCH 102/124] ffv1: convert to refcounted frames.

2013-02-19 Thread Janne Grunau
On 2013-02-19 08:24:14 +0100, Anton Khirnov wrote: > > On Mon, 18 Feb 2013 13:09:37 +0100, Janne Grunau > wrote: > > On 2013-01-08 15:37:54 +0100, Anton Khirnov wrote: > > > --- > > > libavcodec/ffv1.c|5 + > > > libavcodec/ffv1.h|

Re: [libav-devel] [PATCH] zerocodec: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-02-16 10:50:33 +0100, Anton Khirnov wrote: > --- > libavcodec/zerocodec.c | 36 > 1 file changed, 8 insertions(+), 28 deletions(-) > > diff --git a/libavcodec/zerocodec.c b/libavcodec/zerocodec.c > index 8122cca..e503aa7 100644 > --- a/libavcodec/zer

Re: [libav-devel] [PATCH 105/124] cavsdec: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:37:57 +0100, Anton Khirnov wrote: > --- > libavcodec/cavs.c| 18 ++ > libavcodec/cavsdec.c | 37 ++--- > 2 files changed, 24 insertions(+), 31 deletions(-) > > diff --git a/libavcodec/cavs.c b/libavcodec/cavs.c > index c1108

Re: [libav-devel] [PATCH 104/124] proresdec: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:37:56 +0100, Anton Khirnov wrote: > --- > libavcodec/proresdec.c | 37 ++--- > 1 file changed, 14 insertions(+), 23 deletions(-) > > diff --git a/libavcodec/proresdec.c b/libavcodec/proresdec.c > index f7e32c9..e237f91 100644 > --- a/libavcodec/p

Re: [libav-devel] [PATCH 103/124] dvdec: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:37:55 +0100, Anton Khirnov wrote: > --- > libavcodec/dvdec.c | 11 +++ > 1 file changed, 3 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/dvdec.c b/libavcodec/dvdec.c > index bc1bb55..0d301af 100644 > --- a/libavcodec/dvdec.c > +++ b/libavcodec/dvdec.c > @@ -

Re: [libav-devel] [PATCH 118/124] utvideodec: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:38:10 +0100, Anton Khirnov wrote: > --- > libavcodec/utvideo.h|1 - > libavcodec/utvideodec.c | 46 +++--- > 2 files changed, 19 insertions(+), 28 deletions(-) > > diff --git a/libavcodec/utvideo.h b/libavcodec/utvideo.h > index 0

Re: [libav-devel] [PATCH 121/124] lagarith: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:38:13 +0100, Anton Khirnov wrote: > --- > libavcodec/lagarith.c | 21 ++--- > 1 file changed, 6 insertions(+), 15 deletions(-) > > diff --git a/libavcodec/lagarith.c b/libavcodec/lagarith.c > index 33dd8b0..9d84d26 100644 > --- a/libavcodec/lagarith.c > +++ b/li

Re: [libav-devel] [PATCH 086/124] eatgv: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-02-16 14:54:18 +0100, Anton Khirnov wrote: > > On Fri, 15 Feb 2013 14:43:57 +0100, Janne Grunau > wrote: > > On 2013-01-08 15:37:38 +0100, Anton Khirnov wrote: > > > Additionally switch to get_bufer() and enable DR1. > > > > > @@ -292,46

Re: [libav-devel] [PATCH] cdgraphics: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-02-16 21:23:25 +0100, Anton Khirnov wrote: > --- > libavcodec/cdgraphics.c | 64 > ++- > 1 file changed, 30 insertions(+), 34 deletions(-) > > diff --git a/libavcodec/cdgraphics.c b/libavcodec/cdgraphics.c > index 9f402ca..99257c1 100644 > --

Re: [libav-devel] [PATCH 124/124] vp3: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:38:16 +0100, Anton Khirnov wrote: > --- > libavcodec/vp3.c | 177 > -- > 1 file changed, 105 insertions(+), 72 deletions(-) ok Janne ___ libav-devel mailing list libav-devel@libav.o

Re: [libav-devel] [PATCH 102/124] ffv1: convert to refcounted frames.

2013-02-18 Thread Janne Grunau
On 2013-01-08 15:37:54 +0100, Anton Khirnov wrote: > --- > libavcodec/ffv1.c|5 + > libavcodec/ffv1.h|2 ++ > libavcodec/ffv1dec.c | 48 ++-- > 3 files changed, 25 insertions(+), 30 deletions(-) > > diff --git a/libavcodec/ffv1.c b

Re: [libav-devel] [PATCH 123/124] vp8: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:38:15 +0100, Anton Khirnov wrote: > --- > libavcodec/vp8.c | 204 > ++ > libavcodec/vp8.h | 27 > 2 files changed, 128 insertions(+), 103 deletions(-) > > diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c > index 3b8

Re: [libav-devel] [PATCH 099/124] escape124: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:51 +0100, Anton Khirnov wrote: > --- > libavcodec/escape124.c | 28 +++- > 1 file changed, 15 insertions(+), 13 deletions(-) > > diff --git a/libavcodec/escape124.c b/libavcodec/escape124.c > index a27ab68..b304dd8 100644 > --- a/libavcodec/escape124.

Re: [libav-devel] [PATCH 098/124] pngdec: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:50 +0100, Anton Khirnov wrote: > --- > libavcodec/pngdec.c | 39 +++ > 1 file changed, 15 insertions(+), 24 deletions(-) > > diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c > index c4e7ebc..a5f83c9 100644 > --- a/libavcodec/pngdec.c

Re: [libav-devel] [PATCH 097/124] cdgraphics: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:49 +0100, Anton Khirnov wrote: > --- > libavcodec/cdgraphics.c | 59 > ++- > 1 file changed, 27 insertions(+), 32 deletions(-) > > diff --git a/libavcodec/cdgraphics.c b/libavcodec/cdgraphics.c > index e4ed83b..9d54626 100644 > --

Re: [libav-devel] [PATCH 095/124] msvideo1: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:47 +0100, Anton Khirnov wrote: > --- > libavcodec/msvideo1.c | 15 --- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/msvideo1.c b/libavcodec/msvideo1.c > index 860a893..d0aee12 100644 > --- a/libavcodec/msvideo1.c > +++ b/libavcode

Re: [libav-devel] [PATCH 094/124] sgidec: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:46 +0100, Anton Khirnov wrote: > --- > libavcodec/sgidec.c | 32 ++-- > 1 file changed, 2 insertions(+), 30 deletions(-) > > diff --git a/libavcodec/sgidec.c b/libavcodec/sgidec.c > index 0e72751..a4b3128 100644 > --- a/libavcodec/sgidec.c > +++ b

Re: [libav-devel] [PATCH 093/124] pictor: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:45 +0100, Anton Khirnov wrote: > --- > libavcodec/pictordec.c | 41 +++-- > 1 file changed, 15 insertions(+), 26 deletions(-) > > diff --git a/libavcodec/pictordec.c b/libavcodec/pictordec.c > index 16f9307..322d3d4 100644 > --- a/libavcod

Re: [libav-devel] [PATCH 092/124] smacker: convert to refcounted frames.

2013-02-15 Thread Janne Grunau
On 2013-01-08 15:37:44 +0100, Anton Khirnov wrote: > --- > libavcodec/smacker.c | 15 +++ > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c > index b20a7b6..38e3306 100644 > --- a/libavcodec/smacker.c > +++ b/libavcodec/sm

<    8   9   10   11   12   13   14   15   16   17   >