[FFmpeg-devel] [PATCH] add tag/comment support to the raw flac demuxer

Jason Garrett-Glaser darkshikari
Fri Dec 5 07:26:53 CET 2008


On Thu, Dec 4, 2008 at 10:21 PM, Baptiste Coudurier
<baptiste.coudurier at gmail.com> wrote:
> Jason Garrett-Glaser wrote:
>> On Thu, Dec 4, 2008 at 9:13 PM, Baptiste Coudurier
>> <baptiste.coudurier at gmail.com> wrote:
>>> Jason Garrett-Glaser wrote:
>>>> On Thu, Dec 4, 2008 at 11:06 AM, Baptiste Coudurier
>>>> <baptiste.coudurier at gmail.com> wrote:
>>>>> Hi,
>>>>>
>>>>> 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
>>>> FLAC is used in MKV all the time for HDDVD/Blu-ray rips' raw audio
>>>> (people recompress from DTS-HD/similar)...
>>> "raw audio" ? FLAC is not raw.
>>> HDRIPs' around use AC3 and are forced to keep original bluray's track.
>>
>> Sure, crappy Scene rips use AC3, but not everyone downloads crappy Scene rips.
>>
>> I have seen rips with FLAC audio, recompressed from DTS-HD or LPCM.  Examples:
>>
>> Blood.Diamond.2006.BluRay.1080p.x264.DTS.FLAC.5.1-Crow
>> Memories.1995.Blu-ray.1080p.AC3.FLAC.x264-sJR
>> Sigur R?s Heima 2007 1080i MPEG2 DTS5.1 FLAC2.0-Xorp
>> Underworld Evolution BDrip 1080p MPEG2 FLAC 5.1 MKV
>> House Of Flying Daggers BDrip 1080p MPEG2 FLAC 5.1 MKV
>
> -Crow ? -sJR ? -Xorp ? And you call what crappy scene rips ? LOL

Anyways, I seriously hope you're not arguing in favor of throwing away
support for *files which exist in the wild* just because you don't
like them and you're too lazy to fix the code to work correctly.

I mean, if ffmpeg can support game formats that nobody actually uses,
we can definitely support formats that people *do* use, and do it
correctly.

Dark Shikari




More information about the ffmpeg-devel mailing list