[FFmpeg-devel] [PATCH 1/2] img2: do not use CODEC_ID_RAWVIDEO to trigger processing of independant y/u/v image files

[Måns Rullgård]

Peter Ross <pross at xvid.org> writes:

> On Sat, Feb 12, 2011 at 12:41:26PM +0000, M?ns Rullg?rd wrote:
>> Peter Ross <pross at xvid.org> writes:
>> 
>> > This patch allows img2 to support other CODEC_ID_RAWVIDEO image file formats.
>> > ---
>> > I have split up the original img2-raw patch, to make it more clear. Cheers,
>> 
>> I still don't understand what the patch is supposed to achieve.
>
> Ok.
>
> At present img2 only supports the one RAWVIDEO file type, '.y'.
>
> The '.y' file type is special, in that it each frame consists of three files,
> .y, .u, and .v. As such, img2 has to perform special processing for this type,
> namely opening and closing the u & v files, and extracting planes from
> the yuv420p buffer.
>
> The .y file special processing is triggered when codec_id==CODEC_ID_RAWVIDEO.
> This is fine, for the moment, because img2 only supports one RAWVIDEO file type.
>
> Patch 2/2 adds a second RAWVIDEO filetype. I do not want the special processing
> to happen for '.raw' files. So in Patch 1/1, the trigger condition is changed
> to 'is_y'.

Thanks for the explanation.  Now it makes sense.  I would, however,
prefer a more descriptive name for the new field, perhaps split_planes
or similar.

-- 
M?ns Rullg?rd
mans at mansr.com