On Fri, Nov 15, 2024 at 9:25 PM Anton Khirnov <an...@khirnov.net> wrote:

> Quoting Pavel Koshevoy (2024-11-15 20:32:21)
> > This avoids unpleasant surprises to av_frame_get_buffer callers
> > that explicitly specified 64-byte alignment and didn't get
> > AVFrame.data pointers that are 64-byte aligned.
> >
> > For example, see https://github.com/sekrit-twc/zimg/issues/212
> >
> > Although the zscale issue has already been resolved by other means
> > it would still be prudent to improve the behavior of av_frame_get_buffer
> > to fix any unknown and future instances of similar issues.
> > ---
> >  libavutil/frame.c | 4 +++-
> >  libavutil/frame.h | 7 ++++---
> >  2 files changed, 7 insertions(+), 4 deletions(-)
>
> I think this warrants a micro bump and an APIchanges entry, so external
> callers can reliably depend on this.
>
> Otherwise looks good.
>
> --
> Anton Khirnov
>


Thank you, I've submitted a v3 of the patch with doc/APIchanges entry added
and LIBAVUTIL_VERSION_MICRO bumped to 101.

BTW, I did not make any changes to get_audio_buffer which also accepts
an explicit alignment parameter.  I think it probably needs the same
changes.

Pavel.
_______________________________________________
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".

Reply via email to