From patchwork Mon Feb 1 03:50:49 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jose Da Silva X-Patchwork-Id: 25313 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id BEA86449034 for ; Mon, 1 Feb 2021 05:51:05 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A77F6689A52; Mon, 1 Feb 2021 05:51:05 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mx0-10.i-mecca.net (mx0-10.i-mecca.net [76.74.184.244]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 9F01D6880FF for ; Mon, 1 Feb 2021 05:50:59 +0200 (EET) Received: from mx0.ehosting.ca (localhost [127.0.0.1]) by mx0.i-mecca.net (Postfix) with ESMTP id 40F16161664 for ; Sun, 31 Jan 2021 19:50:56 -0800 (PST) Received: from ns2.i-mecca.net (unknown [192.168.1.2]) by mx0.i-mecca.net (Postfix) with ESMTP id AA32E1614D5 for ; Sun, 31 Jan 2021 19:50:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=joescat.com; s=mail; h=date:from:to:subject; bh=Ly82wxNhNXQI0+GmBPTUrBRstUtyeS9WhrO1il96XY4=; b=ZQnfmRhQiK46eL5G1CXc+8j9pYmYWrt0MMSQhaNte9DbqBd0Scizghnx4vdqpjotYT Rzq2bf9DHiGH8sx1XFxsqFvKfzgWcp+8T+lWFul3w3JYHR8DlWLWOWh8Ytugh41k3V4i dEsC9n6fQnChw35RBzTRmNJPreJVS7qXDwEHC6kd4z5d5uKFJCl4aXg38rQTT2Bzx96F ++BA6uT2gouvVgNbhoqQGXzbeMKAgI0NSeB+qCWIEdLjKu9OAl3AgdbgYxbwJWZGKiHp 0S7F5oH5sqcKLNVfNPatAjFHSkquMpTtuWhO2iFKCZbQi1XT1VhrO8qHJMJ3anDhqqXw m8DA== X-MES: 1.0 X-MM: 1.0 Received: from drived.localnet (d66-183-117-75.bchsia.telus.net [66.183.117.75]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ns2.i-mecca.net (Postfix) with ESMTPSA id 50832300027 for ; Sun, 31 Jan 2021 22:50:54 -0500 (EST) From: Jose Da Silva To: ffmpeg-devel@ffmpeg.org Date: Sun, 31 Jan 2021 19:50:49 -0800 User-Agent: KMail/1.13.7 (Linux/2.6.38.8-desktop586-10.mga; KDE/4.6.5; i686; ; ) MIME-Version: 1.0 Message-Id: <202101311950.49730.digital@joescat.com> Subject: [FFmpeg-devel] [PATCH 1/4] avcodec/xbmenc: substitute end instead of recalculating end X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Minor speed increase, end is calculated before entering parse_str_int(), so let's take advantage of the value and avoid recalculating twice more. This also allows parse_str_int() to work with file size larger than int. Signed-off-by: Joe Da Silva --- libavcodec/xbmdec.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) -- 2.30.0 diff --git a/libavcodec/xbmdec.c b/libavcodec/xbmdec.c index d19bdaee23..2ce70204cf 100644 --- a/libavcodec/xbmdec.c +++ b/libavcodec/xbmdec.c @@ -37,10 +37,8 @@ static int convert(uint8_t x) return x; } -static int parse_str_int(const uint8_t *p, int len, const uint8_t *key) +static int parse_str_int(const uint8_t *p, const uint8_t *end, const uint8_t *key) { - const uint8_t *end = p + len; - for(; ppix_fmt = AV_PIX_FMT_MONOWHITE; end = avpkt->data + avpkt->size; - width = parse_str_int(avpkt->data, avpkt->size, "_width"); - height = parse_str_int(avpkt->data, avpkt->size, "_height"); + width = parse_str_int(avpkt->data, end, "_width"); + height = parse_str_int(avpkt->data, end, "_height"); if ((ret = ff_set_dimensions(avctx, width, height)) < 0) return ret;