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 BBE054F7FC for ; Mon, 23 Jun 2025 13:45:26 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 4D894687C4C; Mon, 23 Jun 2025 16:40:42 +0300 (EEST) Received: from mail-pj1-f52.google.com (mail-pj1-f52.google.com [209.85.216.52]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 43A34680267 for ; Mon, 23 Jun 2025 16:40:40 +0300 (EEST) Received: by mail-pj1-f52.google.com with SMTP id 98e67ed59e1d1-3122a63201bso2978050a91.0 for ; Mon, 23 Jun 2025 06:40:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1750686038; x=1751290838; 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=9cB8gWISadj8vCv0kCKwoNxgqS4RtREUepvSmKJj8xE=; b=FW3+utffKpwsetHm23iLm1AcC9KoKzl2Kvq+N2a0boXvik3mh0Xj3DoGB9jmNUxKet 4BjbQllR8vro2/Pw2Yy3MONQnkyieQ5egNi3wZkrqDL9dpYoNZc5lTwA3AC6BP7fz2IS yTEhS6oltwsKCMIe02V3dlLQKecdlPdV3DlDoNZ9IfNf0r5gOFaa5dO6ove8QbNlv8sq a3HqSepCqugsvZsQrSWzXyUCXKRlWFcz5cg5nY4MBlAy/7BRuqJqIx5kHErRyWfZm24c UA2PWlOXtPGdYVHbTKSKXfzZwIjdR2emuuo9Gb6jnVVf0Po3WBF/qw6wcHdawWB5ou8x f5KQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1750686038; x=1751290838; 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=9cB8gWISadj8vCv0kCKwoNxgqS4RtREUepvSmKJj8xE=; b=ByEhaP5a3OCKKyIJJPFoHbIrktYJNJvkkGBABC9PvUeVu/7KwFq7Cbb1rFE+8wf1FK fL8ACjW4AZK6HYBW7taPKe+OHFqLEeSH/28/TEvwnLvJjllPeryzeYauusbXHWbaVSjV Nv3jMh2vfu9fJ+GSbJDt65kuP0Ptdw9UgyzhegiF5eLEp7+a4aWKVPDapWftMnelO1jw U2kMLKsKz/FbQx6NeA3zv1jeUh5x8CPVgPNw1sAIUdFCklbI2JkJMgp/+CvTmIqkM+nQ RgY1i1zWkCFFsZc5Y6YwzKjETGHX7CsZyh+z48ztcbOG4FhGGOnjUZv5DUDvumIBfyh5 dYBA== X-Gm-Message-State: AOJu0Yzn4IRjrKiKXBORgKwBXigh1bBj9HAdYMZqwRCLZZerSUOnvjmq hSInAETk3sqiTgHcMkHizbjBQASIYL2j4ouUb/00xTMMT0/Gd1yNwPjK1ZM+ag== X-Gm-Gg: ASbGnctuaUuLvTOusOAF1BDDcvwI/iyecPuKLQIRYHx0GtAgynO2WkildBO82rSG5JE SUuGgDfKsyIl6Qy8CyR0vDgw15CYLWDIvxW43vS7iY0k9x6HXH4EGqptk2F7C69s46dtbL0be1F cCfJkFpmyIo05XSyOHcwi87MeSyxyj5yIszunfEHJhFK6NQTuWU6eOk+zCRxCVJ2lDyRkx9JFaG q2jgFqDlGwUu+Y+zw9dRyqdrOLH14zN9DqYwc+uQPvR2eWnqeokANMOQqEiCuSI0+Ov101D6F1+ WVifbknc8ni6Xz6/vKg5stoKfW+Gj+UKi59nwtYtzYYfiyK4phRMUsWsW/9UxjMeUFuVxuBHq3Y g0k3XuNvOYZWlg/6WGq/JA301pkg= X-Google-Smtp-Source: AGHT+IHQBKpZIsdajr2u8tY9S5suihdnIJyq79KhVtrsh280kLlbvLEiL+zvK+DjFob0mIVGHqFhxg== X-Received: by 2002:a17:90b:3c47:b0:312:29e:9ec9 with SMTP id 98e67ed59e1d1-3159d8d9ad9mr21040583a91.24.1750686038180; Mon, 23 Jun 2025 06:40:38 -0700 (PDT) Received: from [127.0.0.1] (master.gitmailbox.com. [34.83.118.50]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-3159df71d33sm8207039a91.8.2025.06.23.06.40.37 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 23 Jun 2025 06:40:37 -0700 (PDT) From: Andreas Rheinhardt X-Google-Original-From: Andreas Rheinhardt Message-Id: In-Reply-To: References: Date: Mon, 23 Jun 2025 13:36:40 +0000 Fcc: Sent MIME-Version: 1.0 To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] [PATCH 40/48] avcodec/h263dec: Use function ptr for decode_picture_header 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 Signed-off-by: Andreas Rheinhardt --- libavcodec/h263dec.c | 27 ++++++++++----------------- libavcodec/h263dec.h | 8 +++----- libavcodec/mpeg4videodec.c | 3 ++- libavcodec/mpeg4videodec.h | 1 - libavcodec/msmpeg4dec.c | 8 +++++--- libavcodec/msmpeg4dec.h | 1 - libavcodec/wmv2dec.c | 3 ++- libavcodec/wmv2dec.h | 1 - 8 files changed, 22 insertions(+), 30 deletions(-) diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index 541eb34789..592a22df0d 100644 --- a/libavcodec/h263dec.c +++ b/libavcodec/h263dec.c @@ -121,6 +121,7 @@ av_cold int ff_h263_decode_init(AVCodecContext *avctx) case AV_CODEC_ID_H263: case AV_CODEC_ID_H263P: avctx->chroma_sample_location = AVCHROMA_LOC_CENTER; + h->decode_header = ff_h263_decode_picture_header; break; case AV_CODEC_ID_MPEG4: break; @@ -144,13 +145,20 @@ av_cold int ff_h263_decode_init(AVCodecContext *avctx) s->h263_pred = 1; s->msmpeg4_version = MSMP4_WMV2; break; - case AV_CODEC_ID_H263I: case AV_CODEC_ID_RV10: case AV_CODEC_ID_RV20: break; +#if CONFIG_H263I_DECODER + case AV_CODEC_ID_H263I: + h->decode_header = ff_intel_h263_decode_picture_header; + break; +#endif +#if CONFIG_FLV_DECODER case AV_CODEC_ID_FLV1: s->h263_flv = 1; + h->decode_header = ff_flv_decode_picture_header; break; +#endif default: av_unreachable("Switch contains a case for every codec using ff_h263_decode_init()"); } @@ -461,22 +469,7 @@ int ff_h263_decode_frame(AVCodecContext *avctx, AVFrame *pict, bak_height = h->c.height; /* let's go :-) */ - if (CONFIG_WMV2_DECODER && h->c.msmpeg4_version == MSMP4_WMV2) { - ret = ff_wmv2_decode_picture_header(h); -#if CONFIG_MSMPEG4DEC - } else if (h->c.msmpeg4_version != MSMP4_UNUSED) { - ret = ff_msmpeg4_decode_picture_header(h); -#endif - } else if (CONFIG_MPEG4_DECODER && avctx->codec_id == AV_CODEC_ID_MPEG4) { - ret = ff_mpeg4_decode_picture_header(h); - } else if (CONFIG_H263I_DECODER && h->c.codec_id == AV_CODEC_ID_H263I) { - ret = ff_intel_h263_decode_picture_header(h); - } else if (CONFIG_FLV_DECODER && h->c.h263_flv) { - ret = ff_flv_decode_picture_header(h); - } else { - ret = ff_h263_decode_picture_header(h); - } - + ret = h->decode_header(h); if (ret < 0 || ret == FRAME_SKIPPED) { if ( h->c.width != bak_width || h->c.height != bak_height) { diff --git a/libavcodec/h263dec.h b/libavcodec/h263dec.h index 0c6228f9d9..2b43dda568 100644 --- a/libavcodec/h263dec.h +++ b/libavcodec/h263dec.h @@ -25,11 +25,6 @@ #include "vlc.h" #include "libavutil/mem_internal.h" -/** - * Return value for header parsers if frame is not coded. - * */ -#define FRAME_SKIPPED 100 - // The defines below define the number of bits that are read at once for // reading vlc values. Changing these may improve speed and data cache needs // be aware though that decreasing them may need the number of stages that is @@ -72,6 +67,9 @@ typedef struct H263DecContext { int rv10_version; ///< RV10 version: 0 or 3 int rv10_first_dc_coded[3]; + int (*decode_header)(struct H263DecContext *const h); +#define FRAME_SKIPPED 100 ///< Frame is not coded + int (*decode_mb)(struct H263DecContext *h); #define SLICE_OK 0 #define SLICE_ERROR -1 diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index c53f5c103a..bc17b0ed09 100644 --- a/libavcodec/mpeg4videodec.c +++ b/libavcodec/mpeg4videodec.c @@ -3736,7 +3736,7 @@ end: return decode_vop_header(ctx, gb, parse_only); } -int ff_mpeg4_decode_picture_header(H263DecContext *const h) +static int mpeg4_decode_picture_header(H263DecContext *const h) { Mpeg4DecContext *const ctx = h263_to_mpeg4(h); @@ -4017,6 +4017,7 @@ static av_cold int decode_init(AVCodecContext *avctx) h->c.h263_pred = 1; h->c.low_delay = 0; /* default, might be overridden in the vol header during header parsing */ + h->decode_header = mpeg4_decode_picture_header; h->decode_mb = mpeg4_decode_mb; ctx->time_increment_bits = 4; /* default value for broken headers */ ctx->quant_precision = 5; diff --git a/libavcodec/mpeg4videodec.h b/libavcodec/mpeg4videodec.h index 337125d4e3..aafde454ea 100644 --- a/libavcodec/mpeg4videodec.h +++ b/libavcodec/mpeg4videodec.h @@ -109,7 +109,6 @@ typedef struct Mpeg4DecContext { int dct_precision; } Mpeg4DecContext; -int ff_mpeg4_decode_picture_header(H263DecContext *const h); int ff_mpeg4_parse_picture_header(Mpeg4DecContext *ctx, GetBitContext *gb, int header, int parse_only); void ff_mpeg4_decode_studio(MpegEncContext *s, uint8_t *dest_y, uint8_t *dest_cb, diff --git a/libavcodec/msmpeg4dec.c b/libavcodec/msmpeg4dec.c index 59ceb7dac6..a96a0335f7 100644 --- a/libavcodec/msmpeg4dec.c +++ b/libavcodec/msmpeg4dec.c @@ -360,7 +360,7 @@ static av_cold void msmpeg4_decode_init_static(void) ff_msmp4_vc1_vlcs_init_once(); } -int ff_msmpeg4_decode_picture_header(H263DecContext *const h) +static int msmpeg4_decode_picture_header(H263DecContext *const h) { MSMP4DecContext *const ms = mpv_to_msmpeg4(h); int code; @@ -445,7 +445,7 @@ int ff_msmpeg4_decode_picture_header(H263DecContext *const h) h->c.inter_intra_pred= 0; break; default: - av_unreachable("ff_msmpeg4_decode_picture_header() only used by MSMP4V1-3, WMV1"); + av_unreachable("msmpeg4_decode_picture_header() only used by MSMP4V1-3, WMV1"); } h->c.no_rounding = 1; if (h->c.avctx->debug & FF_DEBUG_PICT_INFO) @@ -498,7 +498,7 @@ int ff_msmpeg4_decode_picture_header(H263DecContext *const h) ms->bit_rate <= II_BITRATE; break; default: - av_unreachable("ff_msmpeg4_decode_picture_header() only used by MSMP4V1-3, WMV1"); + av_unreachable("msmpeg4_decode_picture_header() only used by MSMP4V1-3, WMV1"); } if (h->c.avctx->debug&FF_DEBUG_PICT_INFO) @@ -847,6 +847,8 @@ av_cold int ff_msmpeg4_decode_init(AVCodecContext *avctx) // We unquantize inter blocks as we parse them. h->c.dct_unquantize_inter = NULL; + h->decode_header = msmpeg4_decode_picture_header; + ff_msmpeg4_common_init(&h->c); switch (h->c.msmpeg4_version) { diff --git a/libavcodec/msmpeg4dec.h b/libavcodec/msmpeg4dec.h index 11a36f2136..e0fb3cf483 100644 --- a/libavcodec/msmpeg4dec.h +++ b/libavcodec/msmpeg4dec.h @@ -53,7 +53,6 @@ extern const VLCElem *ff_mb_non_intra_vlc[4]; extern VLCElem ff_inter_intra_vlc[8]; int ff_msmpeg4_decode_init(AVCodecContext *avctx); -int ff_msmpeg4_decode_picture_header(H263DecContext *const h); int ff_msmpeg4_decode_ext_header(H263DecContext *const h, int buf_size); void ff_msmpeg4_decode_motion(MSMP4DecContext *ms, int *mx_ptr, int *my_ptr); int ff_msmpeg4_decode_block(MSMP4DecContext *ms, int16_t * block, diff --git a/libavcodec/wmv2dec.c b/libavcodec/wmv2dec.c index 4ceae05bdf..53d127d37f 100644 --- a/libavcodec/wmv2dec.c +++ b/libavcodec/wmv2dec.c @@ -201,7 +201,7 @@ static int decode_ext_header(WMV2DecContext *w) return 0; } -int ff_wmv2_decode_picture_header(H263DecContext *const h) +static int wmv2_decode_picture_header(H263DecContext *const h) { int code; @@ -574,6 +574,7 @@ static av_cold int wmv2_decode_init(AVCodecContext *avctx) if ((ret = ff_msmpeg4_decode_init(avctx)) < 0) return ret; + h->decode_header = wmv2_decode_picture_header; h->decode_mb = wmv2_decode_mb; ff_wmv2_common_init(s); diff --git a/libavcodec/wmv2dec.h b/libavcodec/wmv2dec.h index 4dd9359dc9..1bd0d13725 100644 --- a/libavcodec/wmv2dec.h +++ b/libavcodec/wmv2dec.h @@ -24,7 +24,6 @@ #include "mpegvideo.h" struct H263DecContext; -int ff_wmv2_decode_picture_header(struct H263DecContext *const h); int ff_wmv2_decode_secondary_picture_header(struct H263DecContext *const h); void ff_wmv2_add_mb(MpegEncContext *s, int16_t block[6][64], uint8_t *dest_y, uint8_t *dest_cb, uint8_t *dest_cr); -- 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".