Re: [libav-devel] [PATCH v2] rmdec: don't ignore the return value of av_get_packet()

2017-07-16 Thread Luca Barbato
On 7/16/17 6:24 PM, Sean McGovern wrote: > --- > Changes from v1: > * removed av_packet_unref() calls as per Luca's review > --- > libavformat/rmdec.c | 22 +- > 1 file changed, 17 insertions(+), 5 deletions(-) > Looks fine, thank you :)

[libav-devel] [PATCH v2] rmdec: don't ignore the return value of av_get_packet()

2017-07-16 Thread Sean McGovern
--- Changes from v1: * removed av_packet_unref() calls as per Luca's review --- libavformat/rmdec.c | 22 +- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c index 2e93f1d..c666154 100644 --- a/libavformat/rmdec.c +++

[libav-devel] [PATCH 2/2] png: Support RGBA64 pixel format

2017-07-16 Thread Luca Barbato
--- libavcodec/pngdec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 7dc5c283d4..bc8c952352 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -520,6 +520,9 @@ static int decode_frame(AVCodecContext *avctx, }

[libav-devel] [PATCH 1/2] png: Report more details regarding unsupported pixel formats

2017-07-16 Thread Luca Barbato
--- libavcodec/pngdec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 927248f2e3..7dc5c283d4 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -521,6 +521,9 @@ static int decode_frame(AVCodecContext *avctx,

Re: [libav-devel] [PATCH] png: Report more details regarding unsupported pixel formats

2017-07-16 Thread wm4
On Sun, 16 Jul 2017 14:27:50 +0200 Luca Barbato wrote: > On 7/16/17 2:12 PM, wm4 wrote: > > On Sun, 16 Jul 2017 13:46:19 +0200 > > Luca Barbato wrote: > > > >> --- > >> libavcodec/pngdec.c | 2 ++ > >> 1 file changed, 2 insertions(+) > >> > >> diff

Re: [libav-devel] [PATCH] png: Report more details regarding unsupported pixel formats

2017-07-16 Thread Luca Barbato
On 7/16/17 2:12 PM, wm4 wrote: > On Sun, 16 Jul 2017 13:46:19 +0200 > Luca Barbato wrote: > >> --- >> libavcodec/pngdec.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c >> index 927248f2e3..6be463344a 100644 >> ---

Re: [libav-devel] [PATCH] lavf: allow avformat_close_input() with NULL

2017-07-16 Thread Luca Barbato
On 7/16/17 12:43 PM, wm4 wrote: > Behaves more like FFmpeg, makes some API users not crash on exit. > --- > libavformat/utils.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index eaba473914..24a4335016 100644 > ---

Re: [libav-devel] [PATCH] png: Report more details regarding unsupported pixel formats

2017-07-16 Thread wm4
On Sun, 16 Jul 2017 13:46:19 +0200 Luca Barbato wrote: > --- > libavcodec/pngdec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c > index 927248f2e3..6be463344a 100644 > --- a/libavcodec/pngdec.c > +++

[libav-devel] [PATCH] png: Report more details regarding unsupported pixel formats

2017-07-16 Thread Luca Barbato
--- libavcodec/pngdec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 927248f2e3..6be463344a 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -521,6 +521,8 @@ static int decode_frame(AVCodecContext *avctx,

[libav-devel] [PATCH] lavf: allow avformat_close_input() with NULL

2017-07-16 Thread wm4
Behaves more like FFmpeg, makes some API users not crash on exit. --- libavformat/utils.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index eaba473914..24a4335016 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@

[libav-devel] [PATCH v2] imgutils: add function to clear an image to black

2017-07-16 Thread wm4
Black isn't always just memset(ptr, 0, size). Limited YUV in particular requires relatively non-obvious values, and filling a frame with repeating 0 bytes is disallowed in some contexts. With component sizes larger than 8 or packed YUV, this can become relatively complicated. So having a generic