On Mon, May 25, 2015 at 12:43 PM, Hendrik Leppkes <[email protected]> wrote:
> On Mon, May 11, 2015 at 10:52 AM, Steve Lhomme <[email protected]> wrote:
>> diff --git a/libavcodec/dxva2_internal.h b/libavcodec/dxva2_internal.h
>> index b775e6c..23be2c3 100644
>> --- a/libavcodec/dxva2_internal.h
>> +++ b/libavcodec/dxva2_internal.h
>> @@ -28,6 +28,7 @@
>>  #include "config.h"
>>
>>  #include "dxva2.h"
>> +#include "d3d11va.h"
>>  #if HAVE_DXVA_H
>>  #include <dxva.h>
>>  #endif
>> @@ -35,13 +36,34 @@
>>  #include "avcodec.h"
>>  #include "mpegvideo.h"
>>
>> +typedef void DECODER_BUFFER_DESC;
>
> typedef void seems like a slightly odd thing to do.

Well, that's a fake type that's casted to whichever type corresponds
to the dxva2 or d3d11 structure pointer. All instances could be
replaced to void and it would be fine. Adding a name gives more
readability into what's going on IMO.

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

Reply via email to