On Sun, Dec 18, 2011 at 04:24:01PM +0100, Anton Khirnov wrote:
> 
> Author:    Anton Khirnov <[email protected]>
> Committer: Anton Khirnov <[email protected]>
> Date:      Sat Dec 17 15:23:13 2011 +0100
> 
> omadec: split data that will be used in the muxer to a separate file.
> 
> --- /dev/null
> +++ b/libavformat/oma.h
> @@ -0,0 +1,39 @@
> +
> +#ifndef AVFORMAT_OMA_H
> +#define AVFORMAT_OMA_H
> +
> +#define EA3_HEADER_SIZE 96
> +#define ID3v2_EA3_MAGIC "ea3"
> +#define OMA_ENC_HEADER_SIZE 16
> +
> +enum {
> +    OMA_CODECID_ATRAC3  = 0,
> +    OMA_CODECID_ATRAC3P = 1,
> +    OMA_CODECID_MP3     = 3,
> +    OMA_CODECID_LPCM    = 4,
> +    OMA_CODECID_WMA     = 5,
> +};
> +
> +extern const uint16_t ff_oma_srate_tab[6];
> +
> +extern const AVCodecTag ff_oma_codec_tags[];

This is missing #includes for uint16_t and AVCodecTag and thus will not
compile standalone.  Please run 'make checkheaders' at least when you
add new headers.

> +#endif

Please comment the #endif.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to