[Ffmpeg-devel] [PATCH 22/31] Move av_realloc() to the header file.
Michael Niedermayer
michaelni
Sat Mar 10 00:55:30 CET 2007
Hi
On Thu, Mar 08, 2007 at 10:09:05PM +0100, Panagiotis Issaris wrote:
> Hi
>
> 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)
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070310/a712a6ce/attachment.pgp>
More information about the ffmpeg-devel
mailing list