[FFmpeg-devel] [PATCH] Support playing SMV files.

[Stefano Sabatini]

On date Tuesday 2012-10-02 11:50:08 +0100, Ash Hughes encoded:
[...]
>  /**
> + * Copy image plane from src to dst.
> + * That is, copy "height" number of lines of "bytewidth" bytes each.
> + * The first byte of each successive line is separated by *_linesize
> + * bytes. Skips copying the first n lines.
> + *
> + * @param dst_linesize linesize for the image plane in dst
> + * @param src_linesize linesize for the image plane in src
> + * @param nlines number of lines to skip before copying
> + */
> +void av_image_copy_plane_skip(uint8_t       *dst, int dst_linesize,
> +                         const uint8_t *src, int src_linesize,
> +                         int bytewidth, int height, int nlines);

This function seems too much specialized for a generic utils, what
about:

void av_image_copy_plane_slice(uint8_t       *dst, int dst_linesize,
                               const uint8_t *src, int src_linesize,
                               int bytewidth, int y, int h);

That would be consistent with the scaling API, and possibly more
useful in different contexts.

[...]
-- 
FFmpeg = Fundamental and Frightening Mean Pure Exploitable Gnome