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 060804B336 for ; Thu, 4 Jul 2024 18:14:36 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id CDF5368DA1F; Thu, 4 Jul 2024 21:14:33 +0300 (EEST) Received: from alt2.a-painless.mh.aa.net.uk (unknown [81.187.30.51]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 83E2E68D9FC for ; Thu, 4 Jul 2024 21:14:27 +0300 (EEST) Received: from 0.3.a.4.d.8.5.c.f.1.9.b.4.7.0.b.0.5.8.0.9.1.8.0.0.b.8.0.1.0.0.2.ip6.arpa ([2001:8b0:819:850:b074:b91f:c58d:4a30] helo=andrews-2024-laptop.sayers) by painless-a.thn.aa.net.uk with smtp (Exim 4.96) (envelope-from ) id 1sPQyQ-008P4R-1Q for ffmpeg-devel@ffmpeg.org; Thu, 04 Jul 2024 19:14:26 +0100 Date: Thu, 4 Jul 2024 19:14:25 +0100 From: Andrew Sayers To: FFmpeg development discussions and patches Message-ID: References: <20240704143104.1821386-1-ffmpeg@haasn.xyz> <20240704195656.GC4933@haasn.xyz> <20240704195910.GG4933@haasn.xyz> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240704195910.GG4933@haasn.xyz> Subject: Re: [FFmpeg-devel] [PATCH v2 1/8] swscale: document SWS_FULL_CHR_H_* flags 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-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: On Thu, Jul 04, 2024 at 07:59:10PM +0200, Niklas Haas wrote: > On Thu, 04 Jul 2024 19:56:56 +0200 Niklas Haas wrote: > > On Thu, 04 Jul 2024 16:24:24 +0100 Andrew Sayers wrote: > > > On Thu, Jul 04, 2024 at 04:30:57PM +0200, Niklas Haas wrote: > > > > From: Niklas Haas > > > > > > > > Based on my best understanding of what they do, given the source code. > > > > --- > > > > libswscale/swscale.h | 28 ++++++++++++++++++++++++++-- > > > > 1 file changed, 26 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/libswscale/swscale.h b/libswscale/swscale.h > > > > index 9d4612aaf3..e22931cab4 100644 > > > > --- a/libswscale/swscale.h > > > > +++ b/libswscale/swscale.h > > > > @@ -82,11 +82,35 @@ const char *swscale_license(void); > > > > #define SWS_PRINT_INFO 0x1000 > > > > > > > > //the following 3 flags are not completely implemented > > > > -//internal chrominance subsampling info > > > > + > > > > +/** > > > > + * Perform full chroma upsampling when converting to RGB as part of scaling. > > > > > > Nitpick: "as part of scaling" seems redundant - can it be removed? > > > > I wrote it this way because, afaict, this flag does not affect unscaled > > special converters (yuv->rgba). But I can remove it if you still think > > it's unnecessary. > > How about: "Perform full chroma upsampling when upscaling to RGB"? Ah, I hadn't understood that distinction at all. I'd recommend... 1. keep the original if this applies to both up- and down-scaling 2. use the second if it's just for upscaling 3. either way, add a line like this at the end of the section: Note: this flag is ignored by unscaled special converters. I realise this patch is just documenting current behaviour, and I'm not saying that behaviour is correct or incorrect, but it seems important and certainly wasn't intuitive to me. So it's worth mentioning a bit louder :) _______________________________________________ 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".