Message ID | 1588850057-24236-5-git-send-email-lance.lmwang@gmail.com |
---|---|
State | Superseded |
Headers | show |
Series | [FFmpeg-devel,v2,1/5] avcodec/mpegvideo_enc: reindent code | expand |
Context | Check | Description |
---|---|---|
andriy/default | pending | |
andriy/make | success | Make finished |
andriy/make_fate | success | Make fate finished |
On Thu, 7 May 2020, lance.lmwang@gmail.com wrote: > From: Limin Wang <lance.lmwang@gmail.com> > > Signed-off-by: Limin Wang <lance.lmwang@gmail.com> > --- > libavcodec/mpegvideo.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c > index b5ddb1b..8a74a45 100644 > --- a/libavcodec/mpegvideo.c > +++ b/libavcodec/mpegvideo.c > @@ -359,7 +359,7 @@ static int init_duplicate_context(MpegEncContext *s) > int y_size = s->b8_stride * (2 * s->mb_height + 1); > int c_size = s->mb_stride * (s->mb_height + 1); > int yc_size = y_size + 2 * c_size; > - int i; > + int i, ret = AVERROR(ENOMEM); ret is uncecessary. > > if (s->mb_height & 1) > yc_size += 2*s->b8_stride + 2*s->mb_stride; > @@ -408,7 +408,7 @@ static int init_duplicate_context(MpegEncContext *s) > > return 0; > fail: > - return -1; // free() through ff_mpv_common_end() > + return ret; // free() through ff_mpv_common_end() Simply return AVERROR(ENOMEM) because all failures are ENOMEM-s. > } > > static void free_duplicate_context(MpegEncContext *s) > -- Regards, Marton
On Thu, May 07, 2020 at 08:36:50PM +0200, Marton Balint wrote: > > > On Thu, 7 May 2020, lance.lmwang@gmail.com wrote: > > > From: Limin Wang <lance.lmwang@gmail.com> > > > > Signed-off-by: Limin Wang <lance.lmwang@gmail.com> > > --- > > libavcodec/mpegvideo.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c > > index b5ddb1b..8a74a45 100644 > > --- a/libavcodec/mpegvideo.c > > +++ b/libavcodec/mpegvideo.c > > @@ -359,7 +359,7 @@ static int init_duplicate_context(MpegEncContext *s) > > int y_size = s->b8_stride * (2 * s->mb_height + 1); > > int c_size = s->mb_stride * (s->mb_height + 1); > > int yc_size = y_size + 2 * c_size; > > - int i; > > + int i, ret = AVERROR(ENOMEM); > > ret is uncecessary. > > > > > if (s->mb_height & 1) > > yc_size += 2*s->b8_stride + 2*s->mb_stride; > > @@ -408,7 +408,7 @@ static int init_duplicate_context(MpegEncContext *s) > > > > return 0; > > fail: > > - return -1; // free() through ff_mpv_common_end() > > + return ret; // free() through ff_mpv_common_end() > > Simply return AVERROR(ENOMEM) because all failures are ENOMEM-s. Thanks for review, will update the patch. > > > } > > > > static void free_duplicate_context(MpegEncContext *s) > > -- > > Regards, > Marton > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index b5ddb1b..8a74a45 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -359,7 +359,7 @@ static int init_duplicate_context(MpegEncContext *s) int y_size = s->b8_stride * (2 * s->mb_height + 1); int c_size = s->mb_stride * (s->mb_height + 1); int yc_size = y_size + 2 * c_size; - int i; + int i, ret = AVERROR(ENOMEM); if (s->mb_height & 1) yc_size += 2*s->b8_stride + 2*s->mb_stride; @@ -408,7 +408,7 @@ static int init_duplicate_context(MpegEncContext *s) return 0; fail: - return -1; // free() through ff_mpv_common_end() + return ret; // free() through ff_mpv_common_end() } static void free_duplicate_context(MpegEncContext *s)