[FFmpeg-devel] [PATCH] read header in FLAC demuxer

Michael Niedermayer michaelni
Sun Mar 1 05:11:21 CET 2009


On Sat, Feb 28, 2009 at 09:08:18PM -0500, Justin Ruggles wrote:
> Justin Ruggles wrote:
> > Michael Niedermayer wrote:
> >> On Mon, Feb 02, 2009 at 09:00:05PM -0500, Justin Ruggles wrote:
> >>> I'm open to suggestions or other samples to test.
> >> what you tested sounds good
> >> in addition you could test if -acodec copy to containers supporting flac
> >> result in identical files
> > 
> > Good idea. I'll test some, then submit the new patch.
> 
> Now that the FLAC muxer has changed, this patch is somewhat simpler.  It
> still disallows the header to be passed to the decoder with frame data,
> but that is not needed by any of our demuxers after this patch.  One
> thing I'm unsure of is whether or not to bump the lavc minor version
> since the FLAC decoder behavior will change.
> 

> As for "-acodec copy", the only thing that will change is that remuxing
> from raw FLAC to wav or avi will put the STREAMINFO in WAVEFORMATEX like
> it should instead of in the data.  Remuxing raw FLAC to other containers
> is still broken because we don't have a FLAC parser.

will flac in avi files from "old" ffmpeg still work with new?

[...]
-- 
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: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090301/8233f2ec/attachment.pgp>



More information about the ffmpeg-devel mailing list