[FFmpeg-devel] [PATCH] add tag/comment support to the raw flac demuxer
Thu Dec 4 20:30:33 CET 2008
On Thu, Dec 04, 2008 at 11:06:34AM -0800, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > On Mon, Dec 01, 2008 at 02:18:02PM -0800, Jim Radford wrote:
> >> This patch adds support for parsing vorbis comments in plain flac
> >> streams. Only metadata packets are parsed leaving the frame data to
> >> be parsed in raw 1024 byte chunks like before.
> > this wont work with flac in any container short of raw flac.
> > Thus IMO this is unacceptable
> It's not like flac would be put in another container, except ogg and
> this would be a lot worse, and I personnally don't care about flac in ogg.
flac in ogg is idiotic but isnt anyone recording music videos ... with
lossless audio from a cd?
For that use case (and i do not know if anyone is actually using that use
case) flac in some non ogg container would be a possible option
> I guess 90% of flac files are lossless cd audio ripping scene, and in
> this case this patch really adds a feature which can be removed later if
> someone volonteer to do something at codec level.
> This is IMHO acceptable, and I'd like it in svn.
ill review it then
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel