Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-22 Thread Marton Balint
On Wed, 19 Apr 2017, Marton Balint wrote: On Tue, 18 Apr 2017, Michael Niedermayer wrote: On Tue, Apr 18, 2017 at 08:46:24PM +0200, Marton Balint wrote: On Tue, 18 Apr 2017, Michael Niedermayer wrote: On Tue, Apr 18, 2017 at 07:09:30AM +0200, Nicolas George wrote: Le nonidi 29

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-19 Thread Marton Balint
On Tue, 18 Apr 2017, Michael Niedermayer wrote: On Tue, Apr 18, 2017 at 08:46:24PM +0200, Marton Balint wrote: On Tue, 18 Apr 2017, Michael Niedermayer wrote: On Tue, Apr 18, 2017 at 07:09:30AM +0200, Nicolas George wrote: Le nonidi 29 germinal, an CCXXV, Michael Niedermayer a écrit :

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-18 Thread Michael Niedermayer
On Tue, Apr 18, 2017 at 08:46:24PM +0200, Marton Balint wrote: > > > On Tue, 18 Apr 2017, Michael Niedermayer wrote: > > >On Tue, Apr 18, 2017 at 07:09:30AM +0200, Nicolas George wrote: > >>Le nonidi 29 germinal, an CCXXV, Michael Niedermayer a écrit : > +while ((ret =

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-18 Thread Marton Balint
On Tue, 18 Apr 2017, Michael Niedermayer wrote: On Tue, Apr 18, 2017 at 07:09:30AM +0200, Nicolas George wrote: Le nonidi 29 germinal, an CCXXV, Michael Niedermayer a écrit : +while ((ret = avcodec_receive_packet(enc, )) == AVERROR(EAGAIN)) { +ret =

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-18 Thread Michael Niedermayer
On Tue, Apr 18, 2017 at 07:09:30AM +0200, Nicolas George wrote: > Le nonidi 29 germinal, an CCXXV, Michael Niedermayer a écrit : > > > +while ((ret = avcodec_receive_packet(enc, )) == > > > AVERROR(EAGAIN)) { > > > +ret = avcodec_send_frame(enc, NULL); > > The

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-17 Thread Nicolas George
Le nonidi 29 germinal, an CCXXV, Michael Niedermayer a écrit : > > +while ((ret = avcodec_receive_packet(enc, )) == > > AVERROR(EAGAIN)) { > > +ret = avcodec_send_frame(enc, NULL); The doc says: # The functions will not return AVERROR(EAGAIN), unless you

Re: [FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-17 Thread Michael Niedermayer
On Sat, Apr 15, 2017 at 08:33:07PM +0200, Marton Balint wrote: > Fixes Coverity CID 1404841. > > Signed-off-by: Marton Balint > --- > ffmpeg.c | 14 +++--- > 1 file changed, 11 insertions(+), 3 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index e4b94b2..dd179b0

[FFmpeg-devel] [PATCH] ffmpeg; check return code of avcodec_send_frame when flushing encoders

2017-04-15 Thread Marton Balint
Fixes Coverity CID 1404841. Signed-off-by: Marton Balint --- ffmpeg.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index e4b94b2..dd179b0 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -1904,8 +1904,6 @@ static void