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

[Ivo]

On Monday 28 January 2008 02:22, Compn wrote:
> 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:
> >> >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.

I chose to put the whole message in the comment instead of extra README 
files. As this subject has come up several times already and nothing was 
done, I took the liberty to add it myself :) Now let the flames commence ;)

--Ivo