[FFmpeg-devel] [PATCH] Move cavs dsp functions to their own struct

Måns Rullgård mans
Tue Aug 3 22:59:45 CEST 2010


M?ns Rullg?rd <mans at mansr.com> writes:

> Stefan Gehrer <stefan.gehrer at gmx.de> writes:
>
>> On 08/03/2010 01:13 AM, Mans Rullgard wrote:
>>
>>> diff --git a/libavcodec/cavs.h b/libavcodec/cavs.h
>>> index 729c83e..c80fd32 100644
>>> --- a/libavcodec/cavs.h
>>> +++ b/libavcodec/cavs.h
>>> @@ -24,6 +24,7 @@
>>>
>>>   #include "dsputil.h"
>>>   #include "mpegvideo.h"
>>> +#include "cavsdsp.h"
>>>
>>>   #define SLICE_MAX_START_CODE    0x000001af
>>>   #define EXT_START_CODE          0x000001b5
>>> @@ -153,6 +154,7 @@ struct dec_2dvlc {
>>>
>>>   typedef struct {
>>>       MpegEncContext s;
>>> +    CAVSDSPContext dsp;
>>
>> Since there are still references to the DSPContext left
>> (e.g. clear_block and h264 chroma MC) I agree with Michael that it
>> is a bit confusing, renaming to e.g. "cdsp" would be sufficient to
>> avoid it.
>
> OK, will do.

Done.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list