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

Aurelien Jacobs aurel
Mon Mar 12 16:18:59 CET 2007


On Mon, 12 Mar 2007 15:50:35 +0100
Panagiotis Issaris <takis.issaris at uhasselt.be> wrote:

> -----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"?

common.h is installed and it #include mem.h so installed common.h won't
work until you install mem.h.
So I think that mem.h must be installed.

Aurel




More information about the ffmpeg-devel mailing list