Hi Ethan
On Sat, Jun 07, 2025 at 12:15:06PM -0500, Ethan Halsall wrote:
> Hi Michael,
[...]
> @@ -201,8 +203,14 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> *inpic)
>
> for (i = 0; i < 3; i++) {
> int is_chroma = !!i;
> - int w = AV_CEIL_RSHIFT(inlink->w, is_chroma);
> - int h = AV_CEIL_RSHIFT(inlink->h, is_chroma);
> + int w, h;
> + if (is_chroma) {
> + w = inlink->w >> pix_fmt_desc->log2_chroma_w;
> + h = inlink->h >> pix_fmt_desc->log2_chroma_h;why is this not using AV_CEIL_RSHIFT() ? thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB What is money laundering? Its paying someone and not telling the government.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
