Diego Biurrun <[email protected]> writes:

> ---
> This could possibly be considered an API break, but the macro looked
> obscure enough to ignore this for the moment.
>
>  libavutil/avutil.h |    2 --
>  libavutil/mem.h    |    4 +++-
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavutil/avutil.h b/libavutil/avutil.h
> index 351e827..391f695 100644
> --- a/libavutil/avutil.h
> +++ b/libavutil/avutil.h
> @@ -121,8 +121,6 @@
>  #define AV_GLUE(a, b) a ## b
>  #define AV_JOIN(a, b) AV_GLUE(a, b)
>
> -#define AV_PRAGMA(s) _Pragma(#s)
> -
>  /**
>   * @}
>   */
> diff --git a/libavutil/mem.h b/libavutil/mem.h
> index cd8490b..9813ef7 100644
> --- a/libavutil/mem.h
> +++ b/libavutil/mem.h
> @@ -26,14 +26,16 @@
>  #ifndef AVUTIL_MEM_H
>  #define AVUTIL_MEM_H
>
> +#include <sys/types.h>
> +
>  #include "attributes.h"
> -#include "avutil.h"
>
>  /**
>   * @addtogroup lavu_mem
>   * @{
>   */
>
> +#define AV_PRAGMA(s) _Pragma(#s)
>
>  #if defined(__ICC) && _ICC < 1200 || defined(__SUNPRO_C)
>      #define DECLARE_ALIGNED(n,t,v)      t __attribute__ ((aligned (n))) v
> -- 

Rejected.  That's the only place it is used *now*, but it is a very
generic thing that may come in handy elsewhere in the future.  Please
leave it where it is.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to