[Ffmpeg-devel] [PATCH 22/31] Move av_realloc() to the header file.

Panagiotis Issaris takis.issaris
Mon Mar 12 15:50:35 CET 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi

Michael Niedermayer schreef:
> On Thu, Mar 08, 2007 at 10:09:05PM +0100, Panagiotis Issaris wrote:
>> Michael Niedermayer schreef:
>>> On Mon, Mar 05, 2007 at 04:28:47PM +0100, Panagiotis Issaris wrote:
>>>> Michael Niedermayer schreef:
>>>>> On Sun, Mar 04, 2007 at 11:11:47PM +0100, takis.issaris at uhasselt.be wrote:
>>>>>> Move libavutil/mem.c public API comments to common.h. The moved comments are
>>>>>> unmodified.
>>>>> i think mem.c should get its own mem.h besides this ive no
>>>>> objections to moving the comments to the header, if you like you can
>>>>> move all of the mem.c comments to common.h and later move them to mem.h
>>>> Should common.h include mem.h or should all files that need the memory
>>>> allocation functions directly include mem.h (that would be 141 files)?
>>> due to API compatibility common.h needs to #inlude it
>>> [...]
>> The attached patch moves the memory related function out of common.h
>> into their own header file mem.h.
>>
>>  common.h |   42 +---------------------------------------
>>  mem.h    |   65
>> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 67 insertions(+), 40 deletions(-)
> 
> looks ok, and dont forget svn cp! (and check first that you didnt add any
> cosmetic chanegs by mistake (diff old common.h against mem.h)
> 
> [...]
Applied.

Should mem.h be installed on "make install"?

Patch attached to add mem.h to the publicly available header files.

 Makefile |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

With friendly regards,
Takis
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFF9Wi79kOxLuzz4CkRAsscAJ9rbYV1E01wL5JiFENdV6Ey2VtT9wCfQLK8
ciP6vqV+PHQTEds7y2B5KMo=
=7VGX
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20070312T154936-ffmpeg-install_mem_h.diff
Type: text/x-patch
Size: 414 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070312/7a5b14ba/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20070312T154936-ffmpeg-install_mem_h.diff.sig
Type: application/octet-stream
Size: 65 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070312/7a5b14ba/attachment.obj>



More information about the ffmpeg-devel mailing list