[MPlayer-dev-eng] [PATCH] support SN9C1xx web camera

[Compn]

On Mon, 28 Jan 2008 01:42:10 +0100, Ivo wrote:

>On Monday 28 January 2008 01:36, Compn wrote:
>> On Sat, 26 Jan 2008 22:05:01 +0100, Reimar Döffinger wrote:
>> >Hello,
>> >
>> >On Sat, Jan 26, 2008 at 09:30:30PM +0100, Ötvös Attila wrote:
>> >> I make support Sequential RGB Bayer and SN9C1xx compression to
>> >> v4l2.
>> >
>> >Please also proved sample files.
>> >
>> >> Index: libmpcodecs/vd_bayer.c
>> >
>> >I don't think we should accept new video decoders, they belong into
>> >libavcodec.
>>
>> again i must STRESS that someone write down this information in the
>> source and docs before we get more patches for depreciated parts of
>> mplayer.
>
>I think the best way would be to put a README in libmpcodecs and
>libmpdemux and add a line to vd.c, ve.c, muxer.c and demuxer.c reading:
>
>/* Before adding a new X, read the README file in this directory */
>
>X being (de|en)coder or (de)muxer. Or perhaps a line saying none
>should be added and a new X should be implemented for lav*. Whatever
>the line will be, it can be placed at the end of the X_list[] where
>people would reference their new X.

i agree, the /* comment */ should be placed in the list where a
developer would add his new file.

-compn