diff mbox

[FFmpeg-devel,4/4] avcodec/vp8: remove redundant check

Message ID 20170307014143.19735-4-michael@niedermayer.cc
State Accepted
Commit 5098a6f6275a57f122cd8f03e7ffbe5dd090b8e0
Headers show

Commit Message

Michael Niedermayer March 7, 2017, 1:41 a.m. UTC 
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 libavcodec/vp8.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Paul B Mahol March 7, 2017, 8:20 a.m. UTC | #1 
On 3/7/17, Michael Niedermayer <michael@niedermayer.cc> wrote:
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  libavcodec/vp8.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c
> index 8039e7817e..a3d057d62e 100644
> --- a/libavcodec/vp8.c
> +++ b/libavcodec/vp8.c
> @@ -2504,8 +2504,6 @@ int vp78_decode_mb_row_sliced(AVCodecContext *avctx,
> void *tdata, int jobnr,
>
>      td->thread_nr = threadnr;
>      for (mb_y = jobnr; mb_y < s->mb_height; mb_y += num_jobs) {
> -        if (mb_y >= s->mb_height)
> -            break;
>          td->thread_mb_pos = mb_y << 16;
>          ret = s->decode_mb_row_no_filter(avctx, tdata, jobnr, threadnr);
>          if (ret < 0)
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

ok
Michael Niedermayer March 7, 2017, 5:44 p.m. UTC | #2 
On Tue, Mar 07, 2017 at 09:20:09AM +0100, Paul B Mahol wrote:
> On 3/7/17, Michael Niedermayer <michael@niedermayer.cc> wrote:
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >  libavcodec/vp8.c | 2 --
> >  1 file changed, 2 deletions(-)
> >
> > diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c
> > index 8039e7817e..a3d057d62e 100644
> > --- a/libavcodec/vp8.c
> > +++ b/libavcodec/vp8.c
> > @@ -2504,8 +2504,6 @@ int vp78_decode_mb_row_sliced(AVCodecContext *avctx,
> > void *tdata, int jobnr,
> >
> >      td->thread_nr = threadnr;
> >      for (mb_y = jobnr; mb_y < s->mb_height; mb_y += num_jobs) {
> > -        if (mb_y >= s->mb_height)
> > -            break;
> >          td->thread_mb_pos = mb_y << 16;
> >          ret = s->decode_mb_row_no_filter(avctx, tdata, jobnr, threadnr);
> >          if (ret < 0)
> > --
> > 2.11.0
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> 
> ok

applied
will be in my next push

thx

[...]
diff mbox

Patch

diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c
index 8039e7817e..a3d057d62e 100644
--- a/libavcodec/vp8.c
+++ b/libavcodec/vp8.c
@@ -2504,8 +2504,6 @@  int vp78_decode_mb_row_sliced(AVCodecContext *avctx, void *tdata, int jobnr,
 
     td->thread_nr = threadnr;
     for (mb_y = jobnr; mb_y < s->mb_height; mb_y += num_jobs) {
-        if (mb_y >= s->mb_height)
-            break;
         td->thread_mb_pos = mb_y << 16;
         ret = s->decode_mb_row_no_filter(avctx, tdata, jobnr, threadnr);
         if (ret < 0)