[FFmpeg-devel] [PATCH 1/3] lavu/frame: add av_frame_check_align().

Hendrik Leppkes h.leppkes at gmail.com
Thu May 18 11:40:25 EEST 2017


On Thu, May 18, 2017 at 10:11 AM, Nicolas George <george at nsup.org> wrote:
> Signed-off-by: Nicolas George <george at nsup.org>
> ---
>  doc/APIchanges    |  3 +++
>  libavutil/frame.c | 17 +++++++++++++++++
>  libavutil/frame.h |  8 ++++++++
>  3 files changed, 28 insertions(+)
>
>

I wonder if there should be an exception in here somewhere for
hardware pixelformats, there is no reason to assume the hardware
pointers passed in AVFrame would be aligned, and aligning them would
cause all sorts of crashes.
Best case from an API standpoint would be if av_frame_check_align can
just claim hardware frames are aligned, so users don't have to
replicate that check.

-  Hendrik


More information about the ffmpeg-devel mailing list