[FFmpeg-devel] [PATCH] vp9: move VP9SharedContext back to the top of VP9Context

Hendrik Leppkes h.leppkes at gmail.com
Tue Sep 19 15:54:06 EEST 2017


On Tue, Sep 19, 2017 at 2:15 PM, Mark Thompson <sw at jkqxz.net> wrote:
> On 19/09/17 09:51, Hendrik Leppkes wrote:
>> VP9SharedContext needs to be the first member so its properties can be
>> safely accessed from hardware accelerators, without the need to share
>> the full VP9Context.
>> ---
>>  libavcodec/vp9dec.h | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavcodec/vp9dec.h b/libavcodec/vp9dec.h
>> index 96c0e43cd2..66573edc79 100644
>> --- a/libavcodec/vp9dec.h
>> +++ b/libavcodec/vp9dec.h
>> @@ -89,8 +89,8 @@ typedef struct VP9Block {
>>  typedef struct VP9TileData VP9TileData;
>>
>>  typedef struct VP9Context {
>> -    VP9TileData *td;
>>      VP9SharedContext s;
>> +    VP9TileData *td;
>>
>>      VP9DSPContext dsp;
>>      VideoDSPContext vdsp;
>>
>
> LGTM.
>
> (You are also fixing #6674 now too.)
>

Pushed.


More information about the ffmpeg-devel mailing list