On Wed, Feb 26, 2014 at 02:28:21AM +0100, Vittorio Giovara wrote:
> ---
> put_bits does not use any mathops functions, but ac3 needs
> the header somewhere.
> Vittorio

Actually, only ac3enc.h needs it.

> --- a/libavcodec/put_bits.h
> +++ b/libavcodec/put_bits.h
> @@ -26,6 +26,8 @@
>  #ifndef AVCODEC_PUT_BITS_H
>  #define AVCODEC_PUT_BITS_H
>  
> +#include "config.h"
> +
>  #include <stdint.h>
>  #include <stdlib.h>
>  #include <assert.h>
> @@ -34,8 +36,6 @@
>  #include "libavutil/common.h"
>  #include "libavutil/intreadwrite.h"
>  #include "libavutil/log.h"
> -#include "mathops.h"
> -#include "config.h"

put_bits.h only needs stdint.h, stddef.h, assert.h and intreadwrite.h.

If you want to untangle this (which I'm happy to see you do) I think
you'll have to be a bit more thorough and remove all unneeded headers.

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

Reply via email to