[FFmpeg-devel] [PATCH] read metadata in FLAC demuxer
Justin Ruggles
justinruggles
Tue Oct 2 05:30:28 CEST 2007
Hi,
Here is a new patch. There is now very little redundancy. The FLAC
demuxer, Ogg-FLAC demuxer, and FLAC decoder all share a single function
to parse the streaminfo header. The rest of the metadata is handled
according to the format and requirements of each. This simplifies the
decoder metadata parsing quite a bit and fixes related bugs.
The changes to the Ogg demuxer are shown here for demonstration purposes
only. The changes would be done separately, in 2 commits. One to
utilize the shared function and another to simplify the code. Due to
the dependency on the flac decoder, this maybe should wait until if/when
a FLAC parser is implemented.
Ideally, I would like to rename flac.c to flacdec.c and put the shared
function into a new flac.c. As far as the build system is concerned,
this would only be useful if there was a parser to use a dependency
reference which is tied to the shared code, similar to how it's done
with AC3.
-Justin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-flac-demuxer-2.diff
Type: text/x-patch
Size: 14176 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071001/41d3535c/attachment.bin>
More information about the ffmpeg-devel
mailing list