[Ffmpeg-devel] [PATCH] install libavformat/riff.h

Michael Niedermayer michaelni
Fri Jan 19 11:00:47 CET 2007


Hi

On Fri, Jan 19, 2007 at 09:04:01AM +0000, M?ns Rullg?rd wrote:
> Nico Sabbi <nicola_sabbi at fastwebnet.it> writes:
> 
> > Anything against $subject?
> > Index: Makefile
> > ===================================================================
> > --- Makefile	(revisione 7386)
> > +++ Makefile	(copia locale)
> > @@ -8,7 +8,7 @@
> >  
> >  OBJS= utils.o cutils.o os_support.o allformats.o
> >  
> > -HEADERS = avformat.h avio.h rtp.h rtsp.h rtspcodes.h
> > +HEADERS = avformat.h avio.h rtp.h rtsp.h rtspcodes.h riff.h
> 
> Rejected for now.  Those RIFF related functions at the very least need
> to be renamed with proper av prefixes before being exported to the
> world. 

nico, just give the ones mplayer needs an av_ prefix and move them to
avformat.h or split riff.h into riff.h+riff_internal.h


> If they are made public, I also think the tables should be
> purged of all non-RIFF codes, of which there still are a few.

this is a seperate thing which has nothing to do with this patch
also for every removed fourcc it has to be demonstrated that the
specific codec cannot be stored in avi _before_ the fourcc is
removed

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20070119/80d17328/attachment.pgp>



More information about the ffmpeg-devel mailing list