Hi,
On Wed, Sep 28, 2022 at 8:09 AM Anton Khirnov <an...@khirnov.net> wrote:
> > if (isBayer(srcFormat)) {
> > + c->dst_slice_align = 2;
>
> IMO it's better to put this next to the line that sets dst_slice_align
> for non-bayer cases, makes it clearer what the final value is.
Are you suggesting setting `dst_slice_align` in a different function?
The way I read `ff_get_unscaled_swscale()` is that it goes through the
quirks of all the different conversions (per source and destination
type). In all cases, it sets the `convert_unscaled` function pointer.
In the cases where there is the need to align (yuv2bgr and
yuv410p_to_yuv[a]420p), it also adds `dst_slice_align`. In the same
fashion, the conversions that affect Bayer sources are set in line
2097.
Thanks,
-Chema
_______________________________________________
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".
