[FFmpeg-devel] [PATCH] RealMedia muxer: support audio codecs other than AC-3

Kostya kostya.shishkov
Sun May 2 13:30:23 CEST 2010


On Sun, May 02, 2010 at 01:10:51PM +0200, Francesco Lavra wrote:
> Hi,
> currently the only audio codec supported by the RealMedia muxer is AC-3.
> This patch series allows muxing audio streams other than AC-3. It works
> at least with the RealAudio 14.4 codec.
> The first and second patches contain simple code refactoring, which is
> used in the third patch (where support for other codecs is actually
> added); lastly, the fourth patch contains cosmetic indentation changes.

Well, having lavf RM muxer support many audio codecs is a good idea,
here are a few comments on your patches.

Patch 1 will not be accepted until you can prove it's not slower that
way.
Making taglist public requires adding "ff_" prefix to its name.
The last patch looks quite fine but I'll leave it to Ronald who's also
quite busy but at least he has means to test it.
 
> Francesco
> 



More information about the ffmpeg-devel mailing list