[Ffmpeg-devel] Re: [PATCH] DVCPRO50 support

Måns Rullgård mru
Fri Mar 3 10:42:37 CET 2006


Baptiste COUDURIER said:
> Roman Shaposhnick wrote:
>> On Fri, Mar 03, 2006 at 01:05:46AM +0100, Baptiste COUDURIER wrote:
>>
>> [...]
>>
>>>IMHO It would simpler to handle wrapping in
>>>different containers (thinking about MOV), which must be different.
>>
>>
>>   Do you have any particular concern in mind ?
>
> Yes, in MOV, DVCPRO50 fourcc is 'd5vn' for NTSC, 'dv5p' for PAL. DVCPRO
> fourcc is 'dvpp' for PAL (yuv420p and 25mbits/s) and DV is 'dvc ' for
> NTSC, 'dvcp' for PAL.
>
>  If I want to fix movenc.c to properly set fourcc, I need to check
> pix_fmt and bitrate and even resolution or framerate, while a different
> codec_id would make me only check for resolution or framerate.

The need for separate codec tags is a peculiarity of mov, not of the codec.
What if some container made the same requirement for MPEG?  Encodings
covered by the same specification, and handled by the same code, should
use the same codec_id.

-- 
M?ns Rullg?rd
mru at inprovide.com





More information about the ffmpeg-devel mailing list