[FFmpeg-devel] [PATCH] Add buffer allocators to VA API support code (take 2)

Michael Niedermayer michaelni
Tue Mar 31 04:00:14 CEST 2009


On Mon, Mar 30, 2009 at 11:41:40PM +0200, Gwenole Beauchesne wrote:
> Le 30 mars 09 ? 23:26, Reimar D?ffinger a ?crit :
>
>> On Mon, Mar 30, 2009 at 11:09:40PM +0200, Gwenole Beauchesne wrote:
>>> This adds some VA API buffers allocation logic so that to get rid of
>>> plain structs in vaapi.h (installable in the future) header. This also
>>> saves a few KBs in struct vaapi_context along the way.
>>
>> Forgot to make alloc_buffer static?
>
> Yes, the initial patch did not have the alloc_buffer() function but rather 
> had it inlined in all allocators. Then cut&paste'd code but forgot to 
> static'fy. ;-)
>
> New patch attached (the same, manually edited).

nice&ok if it works ...
the mere fact that its fewer lines is nice ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is not what we do, but why we do it that matters.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090331/c3054787/attachment.pgp>



More information about the ffmpeg-devel mailing list