| Message ID | 1588850057-24236-1-git-send-email-lance.lmwang@gmail.com |
|---|---|
| State | Accepted |
| Commit | 6825f7c0ba0e2021c47ef13440cefc49e50b0b5f |
| 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, May 07, 2020 at 07:14:13PM +0800, lance.lmwang@gmail.com wrote: > From: Limin Wang <lance.lmwang@gmail.com> > > Signed-off-by: Limin Wang <lance.lmwang@gmail.com> > --- > libavcodec/mpegvideo_enc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c > index b2eb9cf..e1fd92c 100644 > --- a/libavcodec/mpegvideo_enc.c > +++ b/libavcodec/mpegvideo_enc.c > @@ -596,8 +596,8 @@ FF_ENABLE_DEPRECATION_WARNINGS > if ((s->codec_id == AV_CODEC_ID_WMV1 || > s->codec_id == AV_CODEC_ID_WMV2) && > avctx->width & 1) { > - av_log(avctx, AV_LOG_ERROR, "width must be multiple of 2\n"); > - return -1; > + av_log(avctx, AV_LOG_ERROR, "width must be multiple of 2\n"); > + return -1; > } LGTM thx [...]
On Fri, May 08, 2020 at 01:17:54PM +0200, Michael Niedermayer wrote: > On Thu, May 07, 2020 at 07:14:13PM +0800, lance.lmwang@gmail.com wrote: > > From: Limin Wang <lance.lmwang@gmail.com> > > > > Signed-off-by: Limin Wang <lance.lmwang@gmail.com> > > --- > > libavcodec/mpegvideo_enc.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c > > index b2eb9cf..e1fd92c 100644 > > --- a/libavcodec/mpegvideo_enc.c > > +++ b/libavcodec/mpegvideo_enc.c > > @@ -596,8 +596,8 @@ FF_ENABLE_DEPRECATION_WARNINGS > > if ((s->codec_id == AV_CODEC_ID_WMV1 || > > s->codec_id == AV_CODEC_ID_WMV2) && > > avctx->width & 1) { > > - av_log(avctx, AV_LOG_ERROR, "width must be multiple of 2\n"); > > - return -1; > > + av_log(avctx, AV_LOG_ERROR, "width must be multiple of 2\n"); > > + return -1; > > } > > LGTM will apply. > > thx > > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > Democracy is the form of government in which you can choose your dictator > _______________________________________________ > 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_enc.c b/libavcodec/mpegvideo_enc.c index b2eb9cf..e1fd92c 100644 --- a/libavcodec/mpegvideo_enc.c +++ b/libavcodec/mpegvideo_enc.c @@ -596,8 +596,8 @@ FF_ENABLE_DEPRECATION_WARNINGS if ((s->codec_id == AV_CODEC_ID_WMV1 || s->codec_id == AV_CODEC_ID_WMV2) && avctx->width & 1) { - av_log(avctx, AV_LOG_ERROR, "width must be multiple of 2\n"); - return -1; + av_log(avctx, AV_LOG_ERROR, "width must be multiple of 2\n"); + return -1; } if ((s->avctx->flags & (AV_CODEC_FLAG_INTERLACED_DCT | AV_CODEC_FLAG_INTERLACED_ME)) &&