On Mon, May 06, 2013 at 01:12:21PM +0200, Janne Grunau wrote:
> --- a/libavutil/pixdesc.c
> +++ b/libavutil/pixdesc.c
> @@ -1512,3 +1512,51 @@ int av_pix_fmt_count_planes(enum AVPixelFormat pix_fmt)
> +
> +enum AVPixelFormat av_pix_fmt_swap_endianness(enum AVPixelFormat pix_fmt)
> +{
> +#define PIX_FMT_SWAP_ENDIANNESS(be, le)
> \
> + case AV_PIX_FMT_##be: return AV_PIX_FMT_##le;
> \
> + case AV_PIX_FMT_##le: return AV_PIX_FMT_##be
> +
> + switch(pix_fmt) {
> + PIX_FMT_SWAP_ENDIANNESS(GRAY16BE, GRAY16LE);
switch (
Also, I think you should skip the indentation after the
switch, the macro contains case statements after all.
> --- a/libavutil/pixdesc.h
> +++ b/libavutil/pixdesc.h
> @@ -226,4 +226,11 @@ int av_pix_fmt_get_chroma_sub_sample(enum AVPixelFormat
> pix_fmt,
>
> +/**
> + * @return pixel format with swapped endianness if it exists,
> + * otherwise AV_PIX_FMT_NONE.
> + */
> +enum AVPixelFormat av_pix_fmt_swap_endianness(enum AVPixelFormat pix_fmt);
That will generate a Doxygen warning due to an undocumented parameter.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
