[FFmpeg-devel] [PATCH3/3] avfilter YUV aligment 32bytesaligment for Y only and 16 bytes for UV.

[Richard Buteau]

> From: ffmpeg-devel-bounces at mplayerhq.hu [mailto:ffmpeg-devel-
> bounces at mplayerhq.hu] On Behalf Of M?ns Rullg?rd
> Sent: Tuesday, September 21, 2010 2:43 PM
> To: FFmpeg development discussions and patches
> Subject: Re: [FFmpeg-devel] [PATCH3/3] avfilter YUV aligment
> 32bytesaligment for Y only and 16 bytes for UV.
> 
> "Richard Buteau" <rbuteau at rgbnetworks.com> writes:
> 
> >> "Richard Buteau" <rbuteau at rgbnetworks.com> writes:
> >>
> >>> Hi all,
> >>> Some encoder don't support linesize[] that aren't multiple. As an
> >>> example they expect 720/360/360 for yuv image or 732/368/368.
> >>> the avfilter align all linesize on 16 byte, this make a yuv
> 720/368/368.
> >>> 720 isn't a multiple of 368.
> >>> I added alignment on 32bytes for Y only so it's always 732/368/368
> >>
> >> Why?  Nothing requires alignment greater than 16.  If something did,
> >> this patch would be the wrong way to fix it.
> >
> > The other option to preserve 2:1 ratio between linesize[0] and
> > linesize[1]/linesize[2]
> 
> What makes you believe such a requirement exists?
> 
I have no clue if it's a requirement or not. It used to work like that before vf scaler. I assumed it was a bug. My bad. Just point to ffmpeg requirements document I'll go check.

> Also, don't top-post and do quote properly.
> 
> --
> M?ns Rullg?rd
> mans at mansr.com
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel