[FFmpeg-cvslog] r11335 - trunk/libavdevice/v4l.c

[Michael Niedermayer]

On Fri, Dec 28, 2007 at 10:14:29AM +0100, Luca Barbato wrote:
> michael wrote:
> > Author: michael
> > Date: Fri Dec 28 04:30:23 2007
> > New Revision: 11335
> > 
> > Log:
> > Remove AIW support code, it is unfit for ffmpeg svn, doing usleep() and
> > colorspace convertion and deinterlacing in the demuxer. Whoever wants
> > AIW support has to implement this cleanly!
> 
> I agree that this should be removed, but:
> 
> - v4l/v4l2 now have a cleaner way to support AIW?

There never was any need to do deinterlacing or colorspace convertion in
the demuxer. The original patch was bad and i made a serious mistake in
applying it years ago.


> - Removing this that was causes a regression?

I dont have a AIW card to test. Its possible that it works just fine. If
not it has to be implemented cleanly. The messy code was in svn for years
and people did not clean it up ... now they will if its still needed ...
If we leave the code there it will never be cleaned up by anyone ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20071228/0b5e3c1c/attachment.pgp>