This is an automated email from the git hooks/post-receive script.

Git pushed a commit to branch master
in repository ffmpeg.

commit fd77f00a8ff8f0558fef1bf7670c34b009414f2b
Author:     Andreas Rheinhardt <[email protected]>
AuthorDate: Wed Apr 8 09:29:33 2026 +0200
Commit:     Andreas Rheinhardt <[email protected]>
CommitDate: Mon Apr 13 12:46:24 2026 +0200

    avcodec/snow: Avoid always-true branch
    
    The input lines used in ff_snow_inner_add_yblock()
    must always be set (because their values are used).
    The MMX assembly always relied on this.
    
    Signed-off-by: Andreas Rheinhardt <[email protected]>
---
 libavcodec/snow.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libavcodec/snow.c b/libavcodec/snow.c
index b6aa94e1ce..76dcdd5488 100644
--- a/libavcodec/snow.c
+++ b/libavcodec/snow.c
@@ -118,7 +118,6 @@ static av_cold void init_qpel(SnowContext *const s)
 void ff_snow_inner_add_yblock(const uint8_t *obmc, const int obmc_stride, 
uint8_t * * block, int b_w, int b_h,
                               int src_x, int src_y, int src_stride, 
slice_buffer * sb, int add, uint8_t * dst8){
     int y, x;
-    IDWTELEM * dst;
 
     av_assume(add); // add == 0 is currently unused
 
@@ -128,7 +127,9 @@ void ff_snow_inner_add_yblock(const uint8_t *obmc, const 
int obmc_stride, uint8_
         const uint8_t *obmc2= obmc1+ (obmc_stride>>1);
         const uint8_t *obmc3= obmc1+ obmc_stride*(obmc_stride>>1);
         const uint8_t *obmc4= obmc3+ (obmc_stride>>1);
-        dst = slice_buffer_get_line(sb, src_y + y);
+        IDWTELEM *dst = sb->line[src_y+y];
+        av_assert2(dst);
+
         for(x=0; x<b_w; x++){
             int v=   obmc1[x] * block[3][x + y*src_stride]
                     +obmc2[x] * block[2][x + y*src_stride]

_______________________________________________
ffmpeg-cvslog mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to