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

Nico Sabbi nicola_sabbi
Wed Jan 24 00:54:45 CET 2007


Michael Niedermayer wrote:

>>
>>and to riff.h (prepended by ff_):
> 
> 
> av_ prefix ...
> 
> anyway ive thought about this a little and ill fix it myself
> ill add several codec_tag=int<->codec_id=int lists to the AVIn/OutputFormat
> and export nice functions which can map between them AVCodecTag 
> (codec_tag, codec_id currently like CodecTag without invalid_asf) will be
> an opaque struct so user apps wont become dependant on its details
> 
> to get the best codec tag for asf you could then do
> av_codec_get_tag(asf_muxer.codec_tag, codec_id);
> 
> comments welcome but be quick as ill probably have this finished and
> in svn in a few hours ...
> 
> [...]
> 

what about moving the definition of AVCodecTag to avformat.h?
there's nothing riff-specific in it.
in mplayer's demux_lavf codec_wav_tags and codec_bmp_tags can
either be declared as extern (without including riff.h) or
we can add a couple of wrappers, but since they are static const...

-- 
"Without a frontend, mplayer is useless" - someone in mplayer-users
-------------- next part --------------
A non-text attachment was scrubbed...
Name: riff.diff
Type: text/x-patch
Size: 960 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070124/301b3d74/attachment.bin>



More information about the ffmpeg-devel mailing list