Re: [libav-devel] [PATCH] checkasm: v210: Fix array overwrite

2015-09-16 Thread Kieran Kunhya
On 16 September 2015 at 14:03, Henrik Gramner wrote: > --- > tests/checkasm/v210enc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/checkasm/v210enc.c b/tests/checkasm/v210enc.c > index cdb8e76..4f5f6ba 100644 > --- a/tests/checkasm/v210enc.c > +++ b/tests/checkas

[libav-devel] [PATCH] checkasm: v210: Fix array overwrite

2015-09-16 Thread Henrik Gramner
--- tests/checkasm/v210enc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/v210enc.c b/tests/checkasm/v210enc.c index cdb8e76..4f5f6ba 100644 --- a/tests/checkasm/v210enc.c +++ b/tests/checkasm/v210enc.c @@ -43,7 +43,7 @@ AV_WN32A(v0 + i, r);

Re: [libav-devel] [PATCH] d3d11va: WindowsPhone requires a mutex around ID3D11VideoContext

2015-09-16 Thread Hendrik Leppkes
On Wed, Sep 16, 2015 at 1:27 PM, Steve Lhomme wrote: > From: Steve Lhomme > > -- > now with an allocation function for the structure > updated with APIchanges addition (minus an extra line) > --- > doc/APIchanges | 4 > libavcodec/Makefile | 2 +- > libavcodec/d3d11va.c | 33 ++

Re: [libav-devel] [PATCH] d3d11va: WindowsPhone requires a mutex around ID3D11VideoContext

2015-09-16 Thread Steve Lhomme
On Wed, Sep 16, 2015 at 1:24 PM, Steve Lhomme wrote: > From: Steve Lhomme > > -- > now with an allocation function for the structure > updated with APIchanges addition > --- > doc/APIchanges | 5 + > libavcodec/Makefile | 2 +- > libavcodec/d3d11va.c | 33 +++

[libav-devel] [PATCH] d3d11va: WindowsPhone requires a mutex around ID3D11VideoContext

2015-09-16 Thread Steve Lhomme
From: Steve Lhomme -- now with an allocation function for the structure updated with APIchanges addition (minus an extra line) --- doc/APIchanges | 4 libavcodec/Makefile | 2 +- libavcodec/d3d11va.c | 33 + libavcodec/d3d11va.h | 18

Re: [libav-devel] [PATCH] d3d11va: WindowsPhone requires a mutex around ID3D11VideoContext

2015-09-16 Thread Steve Lhomme
On Wed, Sep 16, 2015 at 1:14 PM, Vittorio Giovara wrote: > On Wed, Sep 16, 2015 at 8:19 AM, Steve Lhomme wrote: +/** + * Allocate an AVD3D11VAContext. + * + * @return Newly-allocated AVD3D11VAContext or NULL on failure. + */ +AVD3D11VAContext *av_d3d11va_alloc_contex

[libav-devel] [PATCH] d3d11va: WindowsPhone requires a mutex around ID3D11VideoContext

2015-09-16 Thread Steve Lhomme
From: Steve Lhomme -- now with an allocation function for the structure updated with APIchanges addition --- doc/APIchanges | 5 + libavcodec/Makefile | 2 +- libavcodec/d3d11va.c | 33 + libavcodec/d3d11va.h | 18 -- libavcodec/dxva2.

Re: [libav-devel] [PATCH] d3d11va: WindowsPhone requires a mutex around ID3D11VideoContext

2015-09-16 Thread Vittorio Giovara
On Wed, Sep 16, 2015 at 8:19 AM, Steve Lhomme wrote: >>> +/** >>> + * Allocate an AVD3D11VAContext. >>> + * >>> + * @return Newly-allocated AVD3D11VAContext or NULL on failure. >>> + */ >>> +AVD3D11VAContext *av_d3d11va_alloc_context(void); >>> + >> >> this should be added to doc/APIChanges. > > I

Re: [libav-devel] [PATCH] rtpdec: Forward the memory failure

2015-09-16 Thread Martin Storsjö
On Wed, 16 Sep 2015, Luca Barbato wrote: And avoid a memory leak. --- I need to doublecheck what happens to buf. If we don't do *bufptr, the caller still owns the buffer (and it isn't leaked). So this seems fine to me. In practice it doesn't make much difference though since the return va

Re: [libav-devel] [PATCH V2 1/3] rtpdec: add a trace when jitter buffer is full

2015-09-16 Thread Luca Barbato
On 16/09/15 08:55, Martin Storsjö wrote: > On Tue, 15 Sep 2015, Eloi BAIL wrote: > >> This commit adds a warning trace when jitter buffer >> is full. It helps to understand leading decoding issues. >> >> Signed-off-by: Eloi BAIL >> --- >> libavformat/rtpdec.c | 3 +++ >> 1 file changed, 3 insertio

[libav-devel] [PATCH] rtpdec: Forward the memory failure

2015-09-16 Thread Luca Barbato
And avoid a memory leak. --- I need to doublecheck what happens to buf. libavformat/rtpdec.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c index 0efcdb9..4360bd9 100644 --- a/libavformat/rtpdec.c +++ b/libavformat/rtpde

Re: [libav-devel] [PATCH V2 0/3] rtpdec: add traces about RTP jitter buffer

2015-09-16 Thread Martin Storsjö
On Tue, 15 Sep 2015, Luca Barbato wrote: On 15/09/15 17:38, Eloi BAIL wrote: This patchset adds information about jitter buffer used in rtpdec. The first patch prints as warning when jitter buffer is full. The second patch prints as verbose jitter buffer size set by default in libav headers or