From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id B78EF40598 for ; Fri, 25 Feb 2022 09:13:39 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 2D38368B32F; Fri, 25 Feb 2022 11:13:37 +0200 (EET) Received: from mail8.parnet.fi (mail8.parnet.fi [77.234.108.134]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 0365E689F20 for ; Fri, 25 Feb 2022 11:13:29 +0200 (EET) Received: from mail9.parnet.fi (mail9.parnet.fi [77.234.108.21]) by mail8.parnet.fi with ESMTP id 21P9DSjl020210-21P9DSjm020210 for ; Fri, 25 Feb 2022 11:13:28 +0200 Received: from foo.martin.st (host-97-187.parnet.fi [77.234.97.187]) by mail9.parnet.fi (Postfix) with ESMTPS id D0768A142C for ; Fri, 25 Feb 2022 11:13:28 +0200 (EET) Date: Fri, 25 Feb 2022 11:13:28 +0200 (EET) From: =?ISO-8859-15?Q?Martin_Storsj=F6?= To: FFmpeg development discussions and patches In-Reply-To: <20220224225017.GF2829255@pb2> Message-ID: References: <20220219154831.GW2829255@pb2> <20220223194553.59514-1-martin@martin.st> <20220224225017.GF2829255@pb2> MIME-Version: 1.0 X-FE-Policy-ID: 3:14:2:SYSTEM X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [FFmpeg-devel] [PATCH v2] swscale: Take the destination range into account for yuv->rgb->yuv conversions 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 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Thu, 24 Feb 2022, Michael Niedermayer wrote: > On Wed, Feb 23, 2022 at 09:45:53PM +0200, Martin Storsj=F6 wrote: >> The range parameters need to be set up before calling >> sws_init_context (which selects which fastpaths can be used; >> this gets called by sws_getContext); solely passing them via >> sws_setColorspaceDetails isn't enough. >> >> This fixes producing full range YUV range output when doing >> YUV->YUV conversions between different YUV color spaces. >> >> Signed-off-by: Martin Storsj=F6 >> --- >> libswscale/utils.c | 11 ++++++++--- >> tests/fate/libswscale.mak | 16 ++++++++++++++++ >> tests/ref/fate/sws-yuv-colorspace | 6 ++++++ >> tests/ref/fate/sws-yuv-range | 6 ++++++ >> 4 files changed, 36 insertions(+), 3 deletions(-) >> create mode 100644 tests/ref/fate/sws-yuv-colorspace >> create mode 100644 tests/ref/fate/sws-yuv-range > > LGTM > > tested on mingw32/64 + wine / linux x86 32/64bit, arm/mips qemu Thanks, pushed! // Martin _______________________________________________ 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".