diff mbox series

[FFmpeg-devel,3/5] doc/examples/vaapi_transcode: Simplify loop

Message ID 20240424014522.4015429-3-michael@niedermayer.cc
State Accepted
Commit c9c11a0866d45827201b034349bceb2dc58a3499
Headers show
Series [FFmpeg-devel,1/5] doc/examples/demux_decode: Simplify loop | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished

Commit Message

Michael Niedermayer April 24, 2024, 1:45 a.m. UTC
Fixes: CID1428858(1/2) Logically dead code

Sponsored-by: Sovereign Tech Fund
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 doc/examples/vaapi_transcode.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

mypopy@gmail.com April 24, 2024, 6:39 a.m. UTC | #1
On Wed, Apr 24, 2024 at 9:45 AM Michael Niedermayer
<michael@niedermayer.cc> wrote:
>
> Fixes: CID1428858(1/2) Logically dead code
>
> Sponsored-by: Sovereign Tech Fund
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  doc/examples/vaapi_transcode.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/doc/examples/vaapi_transcode.c b/doc/examples/vaapi_transcode.c
> index 8367cb30404..e1b7a438836 100644
> --- a/doc/examples/vaapi_transcode.c
> +++ b/doc/examples/vaapi_transcode.c
> @@ -215,10 +215,8 @@ static int dec_enc(AVPacket *pkt, const AVCodec *enc_codec)
>
>  fail:
>          av_frame_free(&frame);
> -        if (ret < 0)
> -            return ret;
>      }
> -    return 0;
> +    return ret;
>  }
>
it's ok, thx
>  int main(int argc, char **argv)
> --
> 2.43.2
Michael Niedermayer April 24, 2024, 9:06 p.m. UTC | #2
On Wed, Apr 24, 2024 at 02:39:45PM +0800, mypopy@gmail.com wrote:
> On Wed, Apr 24, 2024 at 9:45 AM Michael Niedermayer
> <michael@niedermayer.cc> wrote:
> >
> > Fixes: CID1428858(1/2) Logically dead code
> >
> > Sponsored-by: Sovereign Tech Fund
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >  doc/examples/vaapi_transcode.c | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/doc/examples/vaapi_transcode.c b/doc/examples/vaapi_transcode.c
> > index 8367cb30404..e1b7a438836 100644
> > --- a/doc/examples/vaapi_transcode.c
> > +++ b/doc/examples/vaapi_transcode.c
> > @@ -215,10 +215,8 @@ static int dec_enc(AVPacket *pkt, const AVCodec *enc_codec)
> >
> >  fail:
> >          av_frame_free(&frame);
> > -        if (ret < 0)
> > -            return ret;
> >      }
> > -    return 0;
> > +    return ret;
> >  }
> >
> it's ok, thx

will apply

thx

[...]ssssssssssss
diff mbox series

Patch

diff --git a/doc/examples/vaapi_transcode.c b/doc/examples/vaapi_transcode.c
index 8367cb30404..e1b7a438836 100644
--- a/doc/examples/vaapi_transcode.c
+++ b/doc/examples/vaapi_transcode.c
@@ -215,10 +215,8 @@  static int dec_enc(AVPacket *pkt, const AVCodec *enc_codec)
 
 fail:
         av_frame_free(&frame);
-        if (ret < 0)
-            return ret;
     }
-    return 0;
+    return ret;
 }
 
 int main(int argc, char **argv)