On Mon, Jul 2, 2012 at 11:08 AM, Diego Biurrun <[email protected]> wrote:
> On Sun, Jul 01, 2012 at 08:48:06PM +0200, Reinhard Tartler wrote:
>> the av_-prefixed attributes must not be defined outside of this file
>> ---
>>  libavutil/attributes.h |   21 +--------------------
>>  1 file changed, 1 insertion(+), 20 deletions(-)
>>
>> --- a/libavutil/attributes.h
>> +++ b/libavutil/attributes.h
>> @@ -32,6 +32,7 @@
>>
>> +// NB: config.h defines this when --enable-small is specified
>>  #ifndef av_always_inline
>>  #if AV_GCC_VERSION_AT_LEAST(3,1)
>>  #    define av_always_inline __attribute__((always_inline)) inline
>
> You can drop this one as well once Mans' patch is in.  So I suggest
> waiting and rebasing on top of that.
>
> Patch looks OK otherwise.

pushed now Måns' patch is now in.

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

Reply via email to