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

[Ivo]

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.

Putting it just in the docs won't cut it I think. If I was to write a new 
decoder for MPlayer, I would not rtfm but rtfs first.

--Ivo