From patchwork Wed Mar 10 15:20:55 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Derek Buitenhuis X-Patchwork-Id: 26311 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 7C56F4494B2 for ; Wed, 10 Mar 2021 19:23:02 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5559268A936; Wed, 10 Mar 2021 19:23:02 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-lj1-f170.google.com (mail-lj1-f170.google.com [209.85.208.170]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 954C36880C3 for ; Wed, 10 Mar 2021 19:22:56 +0200 (EET) Received: by mail-lj1-f170.google.com with SMTP id u18so26589306ljd.3 for ; Wed, 10 Mar 2021 09:22:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=Cb8y72Fh0x5pPAYLB9k8g78Z2EKWOxupHaUdcYKMQUw=; b=YNhbia9nAnYWRJQT/IqJO4KAVkW38LlzXxYAamaZKSA038ARJczx8+QUek1N9kQ3Ag /U5S0rENde6D8iyPebvM55300vlcDickA1gTvkXh8jvYtsC2Hf4eHXpOYJRtoneGwz+2 JjKDmZuzSRyrVcIMdui8GUR0I3JRWmtMwcX8Fe9ZvFQaHM+976bnCQpaW4tRHAFtL1WC vh3Dj9zaQdW6jI4UQpeRvRrkgwJW3x5jcpoq8/ZlShhtCMm+CKapdS88f4MdfJW4Tkur UtsZ3xq1FQ/vRWp1rQuzizGh40BDv1c3QEATQk296ohmLGpEo5uLZe5KYmHWctturqR0 dP3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=Cb8y72Fh0x5pPAYLB9k8g78Z2EKWOxupHaUdcYKMQUw=; b=soWB7fA1r0EDXP9jlBb4DKcM+QDRLLqJm15sI7HgHtj9SXtlImfnrfrNj978cdmyXp I88ZUY30W4Fcmx63gGLV916JOBVfE4txAqHDdzQNS3fJHBPG4tlxVyo52xXgaHGGFHJ7 G48hZsVRFM+KGhxPyp1fI/35DgeNrvMCGSWjKbkQ6M7gjA2lbqcX0v/GKKUl1s54Ellx 4P13zK9h4U18KNPg66oQe+wjXLvaMP8Bu2yIyQtleuQmbB4yGfiHqQJe6WEHpNjU9EXz k6gYr+bHvhXcfIaeKLZ5JjEisRsd+uAvhGfYv4VTpwrzy5iYxltfS3dS+i/ohGvZ7aAJ DJNA== X-Gm-Message-State: AOAM531r1ZHoZSUdaD7WloxaiNPsr3d/CmDMFQVgprzYwoddz150LMNe Jm3j9vsj88eJwrhfLd1UqAUelBukUOY= X-Google-Smtp-Source: ABdhPJzlfuZ6NXPUTl6NBZcbI7r0ruKka3c/oBOAxfr7JuCgcqPdW3HTJRqMimdwxO5pfUps9hCJZw== X-Received: by 2002:a05:6000:114e:: with SMTP id d14mr4088323wrx.236.1615389666559; Wed, 10 Mar 2021 07:21:06 -0800 (PST) Received: from localhost.localdomain ([82.129.110.36]) by smtp.gmail.com with ESMTPSA id f2sm31902499wrq.34.2021.03.10.07.21.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Mar 2021 07:21:06 -0800 (PST) From: Derek Buitenhuis To: ffmpeg-devel@ffmpeg.org Date: Wed, 10 Mar 2021 15:20:55 +0000 Message-Id: <20210310152055.40622-1-derek.buitenhuis@gmail.com> X-Mailer: git-send-email 2.30.0 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH] avcodec/gifenc: Actually use the shrunk palette 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" I have no idea how nobody, including myself, noticed this. Signed-off-by: Derek Buitenhuis --- libavcodec/gif.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/gif.c b/libavcodec/gif.c index 938bc0e928..cafd93e190 100644 --- a/libavcodec/gif.c +++ b/libavcodec/gif.c @@ -383,13 +383,12 @@ static int gif_image_write_image(AVCodecContext *avctx, bytestream_put_le16(bytestream, height); if (palette || !s->use_global_palette) { - const uint32_t *pal = palette ? palette : s->palette; unsigned pow2_count = av_log2(shrunk_palette_count - 1); unsigned i; bytestream_put_byte(bytestream, 1<<7 | pow2_count); /* flags */ for (i = 0; i < 1 << (pow2_count + 1); i++) { - const uint32_t v = pal[i]; + const uint32_t v = shrunk_palette[i]; bytestream_put_be24(bytestream, v); } } else {