From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 3C94B4F1F5 for ; Sun, 15 Jun 2025 17:02:20 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 0089968D993; Sun, 15 Jun 2025 19:58:20 +0300 (EEST) Received: from mail-pf1-f181.google.com (mail-pf1-f181.google.com [209.85.210.181]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 622DD68D91E for ; Sun, 15 Jun 2025 19:58:19 +0300 (EEST) Received: by mail-pf1-f181.google.com with SMTP id d2e1a72fcca58-7489dfb7248so705988b3a.1 for ; Sun, 15 Jun 2025 09:58:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1750006697; x=1750611497; darn=ffmpeg.org; h=cc:to:mime-version:content-transfer-encoding:fcc:subject:date :references:in-reply-to:message-id:from:from:to:cc:subject:date :message-id:reply-to; bh=grCb1sVPiSRwcQdWlV3giYZ3hBL7sNgpRBgIGB46FhU=; b=OuQh96//4ym+tGuxprClKYG+0jWbjAt1VGZoPPvSsUUA9pttjXwA/io68n0KTWpq5J MZUwf2Xhrr6LqzNvdbcZogba7xqAHJehJ5RylRrrUillOGbF37pUUN8FYO2rT0usC4zh YD0fbBtyiKSXrMXpcHZpt515WWY62VNapaKr1Kp7/tf4DskDO9dedUsJ50jI9ZiRQXiR 36go2LJHWVwU6TlO+j2JR9v+6RjiLWaPybJdbni8zoo7dA/Py6VMaMWFbNUVpx5JhA3c ZI+OZq2q/15gOvy5O6+hkn+sePxPAM+NLEBI/Qw6CBG0mHFfjVTAOdfbYi+13rsoBkUR AyPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1750006697; x=1750611497; h=cc:to:mime-version:content-transfer-encoding:fcc:subject:date :references:in-reply-to:message-id:from:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=grCb1sVPiSRwcQdWlV3giYZ3hBL7sNgpRBgIGB46FhU=; b=Y8hgYls+GdFOaKiPCvx4X44h+0+2iub3QZQzHyCGzaNjkgY30P9X+w0vSIE75op1NS d4aO/aPqfGAOX8p0q5rpBQYYW24Lj5L5yvZXpfu7eDm37f5KFfoQZL9ooNeKnEqIPU3+ HlpWTPDGgrDUcJmn6vH6vjhlTL2iLsVmeWI18CAh07wnmG7A4T62shkoyPoG+dSLv9f6 XHCYVTh3P1Yjm93ISpt2lIfo4NZTFQv7QIEU8CbRR5/w0JIpH3G6WWyAzWYW7HkdhpFv GSJuLncAtDOlGsia454kb0HwGcxEyaE7rcWhaYRdk3UhdgNFd+62+zjjZteifoHBJkp3 NBkQ== X-Gm-Message-State: AOJu0YzQR+7QV6iG5L1uajjX8wArnjVvJN8uEAkqpb7cIS/5PMQTomvl gmw64ll3AllsOckMrtPEPJs2owJTCh3gn4pyTzUzJUEYS1RXmB5oyC/1OQpNJA== X-Gm-Gg: ASbGncuPgGga/T/MWfVtLG6ASYN2XofEYpSrF3k5gd/YBbomn+C/pke1zt2GZcGs4n/ hBBzyYvH3Kizqp1NOiFMsDb6iCbsQriaUZH6sxytANbs3eJmCNHt2WveQcAQTn4246cqVDL1cc9 YMJJBNNOUdNwwnvFkvT0iaDscWkCBbB17lGoZ7mAqHrrG59InpsWMrCrMDVHsIRJtQex6oCQqbM vT2cR0PWo/3KMGN2YEzmfeHMaLzYELrTuY0YvImlK7QTJ8nxn8hH9/GC4GDlBwrGIILP9eFY0Bo GFN0kQa97iCKGJFA/e93uM/z0lXSa/E/7jOo5+a32GiLGBf2R6QM8YeF1wmVaXH+v/DTb7+aOCW k6toRBIvsASTGfSz5eektArKmg0I= X-Google-Smtp-Source: AGHT+IHzNSEeq+OE5RqkbELqzx0QItigt+Bo4K/WgTuvapfpfnuMSJeAF7igjF3TgDXbxfEkRov9wg== X-Received: by 2002:aa7:938c:0:b0:746:2b0d:767a with SMTP id d2e1a72fcca58-7489cf9a38bmr7309127b3a.14.1750006697012; Sun, 15 Jun 2025 09:58:17 -0700 (PDT) Received: from [127.0.0.1] (master.gitmailbox.com. [34.83.118.50]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-748900d7340sm5294666b3a.179.2025.06.15.09.58.16 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 15 Jun 2025 09:58:16 -0700 (PDT) From: Andreas Rheinhardt X-Google-Original-From: Andreas Rheinhardt Message-Id: In-Reply-To: References: Date: Sun, 15 Jun 2025 16:54:28 +0000 Fcc: Sent MIME-Version: 1.0 To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH 37/47] avcodec/mpegvideo: Only use a single ac_val pointer X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Andreas Rheinhardt Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: From: Andreas Rheinhardt The chroma pointers ac_val[1] and ac_val[2] are no longer used anywhere. Signed-off-by: Andreas Rheinhardt --- libavcodec/ituh263dec.c | 2 +- libavcodec/mpeg4video.c | 8 +++++--- libavcodec/mpeg4videodec.c | 6 ++---- libavcodec/mpeg4videoenc.c | 4 ++-- libavcodec/mpegvideo.c | 10 +++------- libavcodec/mpegvideo.h | 2 +- libavcodec/mpegvideo_enc.c | 4 ++-- libavcodec/vc1_block.c | 6 +++--- 8 files changed, 19 insertions(+), 23 deletions(-) diff --git a/libavcodec/ituh263dec.c b/libavcodec/ituh263dec.c index 809bfcf564..49a87486cc 100644 --- a/libavcodec/ituh263dec.c +++ b/libavcodec/ituh263dec.c @@ -451,7 +451,7 @@ static void h263_pred_acdc(MpegEncContext * s, int16_t *block, int n) int wrap, a, c, pred_dc, scale; const int xy = s->block_index[n]; int16_t *const dc_val = s->dc_val[0] + xy; - int16_t *const ac_val = (s->ac_val[0] + xy)[0]; + int16_t *const ac_val = (s->ac_val + xy)[0]; /* find prediction */ if (n < 4) { diff --git a/libavcodec/mpeg4video.c b/libavcodec/mpeg4video.c index a690077c58..a53ce72dfd 100644 --- a/libavcodec/mpeg4video.c +++ b/libavcodec/mpeg4video.c @@ -50,11 +50,13 @@ void ff_mpeg4_clean_buffers(MpegEncContext *s) l_xy = (2 * s->mb_y - 1) * l_wrap + s->mb_x * 2 - 1; c_wrap = s->mb_stride; int u_xy = 2 * mb_height * l_wrap + s->mb_y * c_wrap + s->mb_x - 1; + int v_xy = u_xy + c_wrap * (mb_height + 1); + int16_t (*ac_val)[16] = s->ac_val; /* clean AC */ - memset(s->ac_val[0] + l_xy, 0, (l_wrap * 2 + 1) * 16 * sizeof(int16_t)); - memset(s->ac_val[0] + u_xy, 0, (c_wrap + 1) * 16 * sizeof(int16_t)); - memset(s->ac_val[0] + u_xy + c_wrap * (mb_height + 1), 0, (c_wrap + 1) * 16 * sizeof(int16_t)); + memset(ac_val + l_xy, 0, (l_wrap * 2 + 1) * sizeof(*ac_val)); + memset(ac_val + u_xy, 0, (c_wrap + 1) * sizeof(*ac_val)); + memset(ac_val + v_xy, 0, (c_wrap + 1) * sizeof(*ac_val)); /* clean MV */ // we can't clear the MVs as they might be needed by a B-frame diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index 1294138d05..d5d09e875d 100644 --- a/libavcodec/mpeg4videodec.c +++ b/libavcodec/mpeg4videodec.c @@ -326,7 +326,7 @@ void ff_mpeg4_pred_ac(MpegEncContext *s, int16_t *block, int n, int dir) int8_t *const qscale_table = s->cur_pic.qscale_table; /* find prediction */ - ac_val = &s->ac_val[0][0][0] + s->block_index[n] * 16; + ac_val = &s->ac_val[0][0] + s->block_index[n] * 16; ac_val1 = ac_val; if (s->ac_pred) { if (dir == 0) { @@ -3785,9 +3785,7 @@ static av_cold void clear_context(MpegEncContext *s) s->block = NULL; s->blocks = NULL; s->ac_val_base = NULL; - s->ac_val[0] = - s->ac_val[1] = - s->ac_val[2] =NULL; + s->ac_val = NULL; memset(&s->sc, 0, sizeof(s->sc)); s->p_field_mv_table_base = NULL; diff --git a/libavcodec/mpeg4videoenc.c b/libavcodec/mpeg4videoenc.c index 0fa8159f18..543fc9e649 100644 --- a/libavcodec/mpeg4videoenc.c +++ b/libavcodec/mpeg4videoenc.c @@ -130,7 +130,7 @@ static inline void restore_ac_coeffs(MPVEncContext *const s, int16_t block[6][64 memcpy(s->c.block_last_index, zigzag_last_index, sizeof(int) * 6); for (n = 0; n < 6; n++) { - int16_t *ac_val = &s->c.ac_val[0][0][0] + s->c.block_index[n] * 16; + int16_t *ac_val = &s->c.ac_val[0][0] + s->c.block_index[n] * 16; st[n] = s->c.intra_scantable.permutated; if (dir[n]) { @@ -169,7 +169,7 @@ static inline int decide_ac_pred(MPVEncContext *const s, int16_t block[6][64], score -= get_block_rate(s, block[n], s->c.block_last_index[n], s->c.intra_scantable.permutated); - ac_val = &s->c.ac_val[0][0][0] + s->c.block_index[n] * 16; + ac_val = &s->c.ac_val[0][0] + s->c.block_index[n] * 16; ac_val1 = ac_val; if (dir[n]) { const int xy = s->c.mb_x + s->c.mb_y * s->c.mb_stride - s->c.mb_stride; diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index 64563a509b..e528935f68 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -129,9 +129,7 @@ static av_cold int init_duplicate_context(MpegEncContext *s) /* ac values */ if (!FF_ALLOCZ_TYPED_ARRAY(s->ac_val_base, yc_size)) return AVERROR(ENOMEM); - s->ac_val[0] = s->ac_val_base + s->b8_stride + 1; - s->ac_val[1] = s->ac_val_base + y_size + s->mb_stride + 1; - s->ac_val[2] = s->ac_val[1] + c_size; + s->ac_val = s->ac_val_base + s->b8_stride + 1; } return 0; @@ -195,9 +193,7 @@ static void backup_duplicate_context(MpegEncContext *bak, MpegEncContext *src) COPY(start_mb_y); COPY(end_mb_y); COPY(ac_val_base); - COPY(ac_val[0]); - COPY(ac_val[1]); - COPY(ac_val[2]); + COPY(ac_val); #undef COPY } @@ -502,7 +498,7 @@ void ff_clean_intra_table_entries(MpegEncContext *s) dc_val[uxy] = dc_val[vxy] = 1024; /* ac pred */ - int16_t (*ac_val)[16] = s->ac_val[0]; + int16_t (*ac_val)[16] = s->ac_val; av_assume(!((uintptr_t)ac_val & 0xF)); // Don't reset the upper-left luma block, as it will only ever be // referenced by blocks from the same macroblock. diff --git a/libavcodec/mpegvideo.h b/libavcodec/mpegvideo.h index 68d70cc0e3..72614d45f1 100644 --- a/libavcodec/mpegvideo.h +++ b/libavcodec/mpegvideo.h @@ -148,7 +148,7 @@ typedef struct MpegEncContext { uint8_t *coded_block_base; uint8_t *coded_block; ///< used for coded block pattern prediction (msmpeg4v3, wmv1) int16_t (*ac_val_base)[16]; - int16_t (*ac_val[3])[16]; ///< used for MPEG-4 AC prediction, all 3 arrays must be continuous + int16_t (*ac_val)[16]; ///< used for H.263 AIC, MPEG-4 AC prediction int mb_skipped; ///< MUST BE SET only during DECODING uint8_t *mbskip_table; /**< used to avoid copy if macroblock skipped (for black regions for example) and used for B-frame encoding & decoding (contains skip table of next P-frame) */ diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c index 04cebee9c6..870cf7f5a7 100644 --- a/libavcodec/mpegvideo_enc.c +++ b/libavcodec/mpegvideo_enc.c @@ -3327,7 +3327,7 @@ static int encode_thread(AVCodecContext *c, void *arg){ if(storecoefs){ for(i=0; i<6; i++){ dc[i] = s->c.dc_val[0][s->c.block_index[i]]; - memcpy(ac[i], s->c.ac_val[0][s->c.block_index[i]], sizeof(int16_t)*16); + memcpy(ac[i], s->c.ac_val[s->c.block_index[i]], sizeof(*s->c.ac_val)); } } @@ -3337,7 +3337,7 @@ static int encode_thread(AVCodecContext *c, void *arg){ if(storecoefs){ for(i=0; i<6; i++){ s->c.dc_val[0][s->c.block_index[i]] = dc[i]; - memcpy(s->c.ac_val[0][s->c.block_index[i]], ac[i], sizeof(int16_t)*16); + memcpy(s->c.ac_val[s->c.block_index[i]], ac[i], sizeof(*s->c.ac_val)); } } } diff --git a/libavcodec/vc1_block.c b/libavcodec/vc1_block.c index 1661fdd742..9427e96d1e 100644 --- a/libavcodec/vc1_block.c +++ b/libavcodec/vc1_block.c @@ -611,7 +611,7 @@ static int vc1_decode_i_block(VC1Context *v, int16_t block[64], int n, /* Store the quantized DC coeff, used for prediction */ block[0] = dcdiff * s->y_dc_scale; - ac_val = s->ac_val[0][s->block_index[n]]; + ac_val = s->ac_val[s->block_index[n]]; ac_val2 = ac_val; if (dc_pred_dir) // left ac_val -= 16; @@ -750,7 +750,7 @@ static int vc1_decode_i_block_adv(VC1Context *v, int16_t block[64], int n, scale = quant * 2 + ((mquant < 0) ? 0 : v->halfpq); - ac_val = s->ac_val[0][s->block_index[n]]; + ac_val = s->ac_val[s->block_index[n]]; ac_val2 = ac_val; if (dc_pred_dir) // left ac_val -= 16; @@ -938,7 +938,7 @@ static int vc1_decode_intra_block(VC1Context *v, int16_t block[64], int n, if (!a_avail) dc_pred_dir = 1; if (!c_avail) dc_pred_dir = 0; if (!a_avail && !c_avail) use_pred = 0; - ac_val = s->ac_val[0][s->block_index[n]]; + ac_val = s->ac_val[s->block_index[n]]; ac_val2 = ac_val; scale = quant * 2 + ((mquant < 0) ? 0 : v->halfpq); -- ffmpeg-codebot _______________________________________________ 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".